Rename `checkout_submodules.sh` to `submodules_init.sh` with compatibility wrapper by Copilot · Pull Request #686 · mtransitapps/commons

Copilot · 2026-04-17T18:50:21Z

This updates submodule bootstrap naming from checkout_submodules.sh to submodules_init.sh across CI/bootstrap configs. It preserves backward compatibility by keeping checkout_submodules.sh as a fallback entrypoint that delegates to the new script.

Script rename + behavior preservation
- Moved the existing submodule init/update implementation to:
  - shared-overwrite/submodules_init.sh
- Kept the original path:
  - shared-overwrite/checkout_submodules.sh
- Converted the old file into a thin wrapper that forwards args to the new script using exec.
Usage updates
- Repointed all direct invocations from ./checkout_submodules.sh to ./submodules_init.sh in:
  - .github/workflows/mt-dependency-update-baseline.yml
  - .github/workflows/dependency-submission.yml
  - shared-overwrite/.circleci/config.yml
  - shared-overwrite/.travis.yml
- Renamed GitHub Actions step labels from MT check out submodules to MT init submodules in:
  - .github/workflows/mt-dependency-update-baseline.yml
  - .github/workflows/dependency-submission.yml

Compatibility wrapper example

# shared-overwrite/checkout_submodules.sh
# DEPRECATED: use submodules_init.sh
SCRIPT_DIR="$(dirname "$0")";
exec "${SCRIPT_DIR}/submodules_init.sh" "$@";

…apper Agent-Logs-Url: https://github.com/mtransitapps/commons/sessions/e84d359e-83b9-4bd7-b975-5246586f6c8b Co-authored-by: mmathieum <177998+mmathieum@users.noreply.github.com>

mmathieum

@copilot please also replace CI step name MT check out submodules with MT init submodules

Agent-Logs-Url: https://github.com/mtransitapps/commons/sessions/e4a92ad9-cf34-4fba-a577-95af86c2094f Co-authored-by: mmathieum <177998+mmathieum@users.noreply.github.com>

Copilot · 2026-04-17T21:21:23Z

@copilot please also replace CI step name MT check out submodules with MT init submodules

Done in abbb406. I renamed the CI step name to MT init submodules in both workflow files.

gemini-code-assist

Code Review

This pull request renames checkout_submodules.sh to submodules_init.sh and updates the CircleCI and Travis CI configurations to use the new script. The original script is maintained as a deprecated wrapper. Feedback was provided to improve the shell script's robustness by double-quoting variables to prevent word splitting and refining the environment variable check for GitHub Actions.

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

mmathieum · 2026-04-21T15:03:47Z

Test mtransitapps/commons#686 fr-perpignan-sankeo-bus-android#12

…kout-submodules-script

…m:mtransitapps/commons into copilot/rename-checkout-submodules-script

Agent-Logs-Url: https://github.com/mtransitapps/commons/sessions/981b29c5-f6e2-4964-bba4-e0b583d7a644 Co-authored-by: mmathieum <177998+mmathieum@users.noreply.github.com>

…parser': - commons: Build(deps): Bump mtransitapps/gh-actions from 1.1.2 to 1.2.1 in /shared-overwrite/.github/workflows mtransitapps/commons#701 - commons: Modules > image generated > nice to have (2) mtransitapps/commons#710 - commons: Add AdMob release notes URL in comments - commons: GMA Next-Gen SDK migration > OFF mtransitapps/commons#709 - commons: Build(deps): Bump io.netty:netty-codec-http2 from 4.2.6.Final to 4.2.12.Final in /shared mtransitapps/commons#707 - commons: Build(deps): Bump com.gradle.develocity from 4.3.2 to 4.4.1 in /shared mtransitapps/commons#706 - commons: Build(deps): Bump gradle-wrapper from 9.3.1 to 9.4.1 in /shared mtransitapps/commons#705 - commons: Build(deps): Bump com.google.devtools.ksp from 2.3.6 to 2.3.7 in the kotlin-ksp-compose group across 1 directory mtransitapps/commons#704 - commons: Dependabot > + `shared/settings.gradle` mtransitapps/commons#703 - commons: Build(deps): Bump the androidx group with 2 updates mtransitapps/commons#700 - commons: `mt-sync-code-data.yml` > image gen > nice to have mtransitapps/commons#699 - commons: `mt-sync-code-data.yml` > allow image generation for Bike agencies mtransitapps/commons#698 - commons: `mt-record-screenshots.yml` > skip main app mtransitapps/commons#697 - commons: `mt-sync-code-data.yml` > update inputs description for clarity mtransitapps/commons#696 - commons: R8 full mode mtransitapps/commons#691 - commons: Update Glide version from `5.0.5` to `5.0.7` mtransitapps/commons#695 - commons: Rename `checkout_submodules.sh` to `submodules_init.sh` with compatibility wrapper mtransitapps/commons#686 - commons: Prevent `mt-sync-code-data` from triggering release on manual non-default branch runs mtransitapps/commons#694 - commons: CircleCI: set `GIT_LFS_SKIP_SMUDGE` mtransitapps/commons#692 - commons-android: GTFS-RT > Trip Updates > Read from source > use `header.timestamp` mtransitapps/commons-android#125 - commons-android: TimeZone != TimeZoneId mtransitapps/commons-android#124 - commons-android: POI status > readFromSource Instant ext - commons-android: R8 full mode mtransitapps/commons-android#120 - commons-java: Strings cleaner > fix stop name length (default zoom) mtransitapps/commons-java#32 - parser: TimeZone != TimeZoneId mtransitapps/parser#61

…parser': - commons: Build(deps): Bump mtransitapps/gh-actions from 1.1.2 to 1.2.1 in /shared-overwrite/.github/workflows mtransitapps/commons#701 - commons: Modules > image generated > nice to have (2) mtransitapps/commons#710 - commons: Add AdMob release notes URL in comments - commons: GMA Next-Gen SDK migration > OFF mtransitapps/commons#709 - commons: Build(deps): Bump io.netty:netty-codec-http2 from 4.2.6.Final to 4.2.12.Final in /shared mtransitapps/commons#707 - commons: Build(deps): Bump com.gradle.develocity from 4.3.2 to 4.4.1 in /shared mtransitapps/commons#706 - commons: Build(deps): Bump gradle-wrapper from 9.3.1 to 9.4.1 in /shared mtransitapps/commons#705 - commons: Build(deps): Bump com.google.devtools.ksp from 2.3.6 to 2.3.7 in the kotlin-ksp-compose group across 1 directory mtransitapps/commons#704 - commons: Dependabot > + `shared/settings.gradle` mtransitapps/commons#703 - commons: Build(deps): Bump the androidx group with 2 updates mtransitapps/commons#700 - commons: `mt-sync-code-data.yml` > image gen > nice to have mtransitapps/commons#699 - commons: `mt-sync-code-data.yml` > allow image generation for Bike agencies mtransitapps/commons#698 - commons: `mt-record-screenshots.yml` > skip main app mtransitapps/commons#697 - commons: `mt-sync-code-data.yml` > update inputs description for clarity mtransitapps/commons#696 - commons: R8 full mode mtransitapps/commons#691 - commons: Update Glide version from `5.0.5` to `5.0.7` mtransitapps/commons#695 - commons: Rename `checkout_submodules.sh` to `submodules_init.sh` with compatibility wrapper mtransitapps/commons#686 - commons: Prevent `mt-sync-code-data` from triggering release on manual non-default branch runs mtransitapps/commons#694 - commons: CircleCI: set `GIT_LFS_SKIP_SMUDGE` mtransitapps/commons#692 - commons: Protobuf Gradle plugin `0.10.0` and `android.newDsl=true` mtransitapps/commons#690 - commons: Keep PNGs in repo > follow-up to #680 mtransitapps/commons#689 - commons: Keep PNGs in repo to avoid running image gen during all CI mtransitapps/commons#680 - commons: Migrate to GMA Next-Gen SDK mtransitapps/commons#589 - commons: mt-sync-code-data: propagate release store-skip inputs to mt-release trigger mtransitapps/commons#682 - commons: logs++ - commons: logs++ - commons: Revert "Add deleted file `.github/actions/setup/action.yml`" mtransitapps/commons#678 - commons: Add deleted file `.github/actions/setup/action.yml` mtransitapps/commons#676 - commons: Build(deps): Bump com.google.firebase.crashlytics from 3.0.6 to 3.0.7 mtransitapps/commons#671 - commons: Support `MT.DELETE.<filename>` marker files in `shared-overwrite` for file deletion mtransitapps/commons#675 - commons: Build(deps): Bump the gms group with 2 updates mtransitapps/commons#669 - commons: Build(deps): Bump org.xerial:sqlite-jdbc from 3.51.3.0 to 3.53.0.0 mtransitapps/commons#672 - commons: Build(deps): Bump com.google.guava:guava from 33.5.0-android to 33.6.0-android mtransitapps/commons#673 - commons: Build(deps): Bump android-gradlePlugin from 9.1.0 to 9.1.1 mtransitapps/commons#670 - commons: Build(deps): Bump org.jetbrains.kotlinx:kotlinx-serialization-bom from 1.10.0 to 1.11.0 in the kotlin-ksp-compose group mtransitapps/commons#668 - commons: Replace local `setup` composite action by shared repo mtransitapps/commons#667 - commons: Rename `check_data_outdated.sh` → `check_data_not_outdated.sh`, keep wrapper for backward compat mtransitapps/commons#663 - commons: Circle CI: only build `master` branch mtransitapps/commons#665 - commons-android: GTFS-RT > Trip Updates > Read from source > use `header.timestamp` mtransitapps/commons-android#125 - commons-android: TimeZone != TimeZoneId mtransitapps/commons-android#124 - commons-android: POI status > readFromSource Instant ext - commons-android: R8 full mode mtransitapps/commons-android#120 - commons-android: Update SSL cert mtransitapps/commons-android#122 - commons-android: Fix `debug` build != debuggable builds mtransitapps/commons-android#121 - commons-android: Keep PNGs in repo to avoid running image gen during all CI mtransitapps/commons-android#119 - commons-android: clean - commons-java: Strings cleaner > fix stop name length (default zoom) mtransitapps/commons-java#32 - commons-java: Add 'gtfs-prod' to the list of sources mtransitapps/commons-java#31 - parser: TimeZone != TimeZoneId mtransitapps/parser#61 - parser: Fix route provide missing color Java<>JSON config mess mtransitapps/parser#60

…parser': - commons: Build(deps): Bump mtransitapps/gh-actions from 1.1.2 to 1.2.1 in /shared-overwrite/.github/workflows mtransitapps/commons#701 - commons: Modules > image generated > nice to have (2) mtransitapps/commons#710 - commons: Add AdMob release notes URL in comments - commons: GMA Next-Gen SDK migration > OFF mtransitapps/commons#709 - commons: Build(deps): Bump io.netty:netty-codec-http2 from 4.2.6.Final to 4.2.12.Final in /shared mtransitapps/commons#707 - commons: Build(deps): Bump com.gradle.develocity from 4.3.2 to 4.4.1 in /shared mtransitapps/commons#706 - commons: Build(deps): Bump gradle-wrapper from 9.3.1 to 9.4.1 in /shared mtransitapps/commons#705 - commons: Build(deps): Bump com.google.devtools.ksp from 2.3.6 to 2.3.7 in the kotlin-ksp-compose group across 1 directory mtransitapps/commons#704 - commons: Dependabot > + `shared/settings.gradle` mtransitapps/commons#703 - commons: Build(deps): Bump the androidx group with 2 updates mtransitapps/commons#700 - commons: `mt-sync-code-data.yml` > image gen > nice to have mtransitapps/commons#699 - commons: `mt-sync-code-data.yml` > allow image generation for Bike agencies mtransitapps/commons#698 - commons: `mt-record-screenshots.yml` > skip main app mtransitapps/commons#697 - commons: `mt-sync-code-data.yml` > update inputs description for clarity mtransitapps/commons#696 - commons: R8 full mode mtransitapps/commons#691 - commons: Update Glide version from `5.0.5` to `5.0.7` mtransitapps/commons#695 - commons: Rename `checkout_submodules.sh` to `submodules_init.sh` with compatibility wrapper mtransitapps/commons#686 - commons: Prevent `mt-sync-code-data` from triggering release on manual non-default branch runs mtransitapps/commons#694 - commons: CircleCI: set `GIT_LFS_SKIP_SMUDGE` mtransitapps/commons#692 - commons: Protobuf Gradle plugin `0.10.0` and `android.newDsl=true` mtransitapps/commons#690 - commons: Keep PNGs in repo > follow-up to #680 mtransitapps/commons#689 - commons: Keep PNGs in repo to avoid running image gen during all CI mtransitapps/commons#680 - commons: Migrate to GMA Next-Gen SDK mtransitapps/commons#589 - commons: mt-sync-code-data: propagate release store-skip inputs to mt-release trigger mtransitapps/commons#682 - commons-android: GTFS-RT > Trip Updates > Read from source > use `header.timestamp` mtransitapps/commons-android#125 - commons-android: TimeZone != TimeZoneId mtransitapps/commons-android#124 - commons-android: POI status > readFromSource Instant ext - commons-android: R8 full mode mtransitapps/commons-android#120 - commons-android: Update SSL cert mtransitapps/commons-android#122 - commons-android: Fix `debug` build != debuggable builds mtransitapps/commons-android#121 - commons-android: Keep PNGs in repo to avoid running image gen during all CI mtransitapps/commons-android#119 - commons-android: clean - commons-java: Strings cleaner > fix stop name length (default zoom) mtransitapps/commons-java#32 - parser: TimeZone != TimeZoneId mtransitapps/parser#61

…parser': - commons: Build(deps): Bump mtransitapps/gh-actions from 1.1.2 to 1.2.1 in /shared-overwrite/.github/workflows mtransitapps/commons#701 - commons: Modules > image generated > nice to have (2) mtransitapps/commons#710 - commons: Add AdMob release notes URL in comments - commons: GMA Next-Gen SDK migration > OFF mtransitapps/commons#709 - commons: Build(deps): Bump io.netty:netty-codec-http2 from 4.2.6.Final to 4.2.12.Final in /shared mtransitapps/commons#707 - commons: Build(deps): Bump com.gradle.develocity from 4.3.2 to 4.4.1 in /shared mtransitapps/commons#706 - commons: Build(deps): Bump gradle-wrapper from 9.3.1 to 9.4.1 in /shared mtransitapps/commons#705 - commons: Build(deps): Bump com.google.devtools.ksp from 2.3.6 to 2.3.7 in the kotlin-ksp-compose group across 1 directory mtransitapps/commons#704 - commons: Dependabot > + `shared/settings.gradle` mtransitapps/commons#703 - commons: Build(deps): Bump the androidx group with 2 updates mtransitapps/commons#700 - commons: `mt-sync-code-data.yml` > image gen > nice to have mtransitapps/commons#699 - commons: `mt-sync-code-data.yml` > allow image generation for Bike agencies mtransitapps/commons#698 - commons: `mt-record-screenshots.yml` > skip main app mtransitapps/commons#697 - commons: `mt-sync-code-data.yml` > update inputs description for clarity mtransitapps/commons#696 - commons: R8 full mode mtransitapps/commons#691 - commons: Update Glide version from `5.0.5` to `5.0.7` mtransitapps/commons#695 - commons: Rename `checkout_submodules.sh` to `submodules_init.sh` with compatibility wrapper mtransitapps/commons#686 - commons: Prevent `mt-sync-code-data` from triggering release on manual non-default branch runs mtransitapps/commons#694 - commons: CircleCI: set `GIT_LFS_SKIP_SMUDGE` mtransitapps/commons#692 - commons-android: GTFS-RT > Trip Updates > Read from source > use `header.timestamp` mtransitapps/commons-android#125 - commons-android: TimeZone != TimeZoneId mtransitapps/commons-android#124 - commons-android: POI status > readFromSource Instant ext - commons-android: R8 full mode mtransitapps/commons-android#120 - commons-java: Strings cleaner > fix stop name length (default zoom) mtransitapps/commons-java#32 - parser: TimeZone != TimeZoneId mtransitapps/parser#61

…parser': - commons: Build(deps): Bump mtransitapps/gh-actions from 1.1.2 to 1.2.1 in /shared-overwrite/.github/workflows mtransitapps/commons#701 - commons: Modules > image generated > nice to have (2) mtransitapps/commons#710 - commons: Add AdMob release notes URL in comments - commons: GMA Next-Gen SDK migration > OFF mtransitapps/commons#709 - commons: Build(deps): Bump io.netty:netty-codec-http2 from 4.2.6.Final to 4.2.12.Final in /shared mtransitapps/commons#707 - commons: Build(deps): Bump com.gradle.develocity from 4.3.2 to 4.4.1 in /shared mtransitapps/commons#706 - commons: Build(deps): Bump gradle-wrapper from 9.3.1 to 9.4.1 in /shared mtransitapps/commons#705 - commons: Build(deps): Bump com.google.devtools.ksp from 2.3.6 to 2.3.7 in the kotlin-ksp-compose group across 1 directory mtransitapps/commons#704 - commons: Dependabot > + `shared/settings.gradle` mtransitapps/commons#703 - commons: Build(deps): Bump the androidx group with 2 updates mtransitapps/commons#700 - commons: `mt-sync-code-data.yml` > image gen > nice to have mtransitapps/commons#699 - commons: `mt-sync-code-data.yml` > allow image generation for Bike agencies mtransitapps/commons#698 - commons: `mt-record-screenshots.yml` > skip main app mtransitapps/commons#697 - commons: `mt-sync-code-data.yml` > update inputs description for clarity mtransitapps/commons#696 - commons: R8 full mode mtransitapps/commons#691 - commons: Update Glide version from `5.0.5` to `5.0.7` mtransitapps/commons#695 - commons: Rename `checkout_submodules.sh` to `submodules_init.sh` with compatibility wrapper mtransitapps/commons#686 - commons: Prevent `mt-sync-code-data` from triggering release on manual non-default branch runs mtransitapps/commons#694 - commons: CircleCI: set `GIT_LFS_SKIP_SMUDGE` mtransitapps/commons#692 - commons: Protobuf Gradle plugin `0.10.0` and `android.newDsl=true` mtransitapps/commons#690 - commons: Keep PNGs in repo > follow-up to #680 mtransitapps/commons#689 - commons: Keep PNGs in repo to avoid running image gen during all CI mtransitapps/commons#680 - commons: Migrate to GMA Next-Gen SDK mtransitapps/commons#589 - commons: mt-sync-code-data: propagate release store-skip inputs to mt-release trigger mtransitapps/commons#682 - commons: logs++ - commons: logs++ - commons: Revert "Add deleted file `.github/actions/setup/action.yml`" mtransitapps/commons#678 - commons: Add deleted file `.github/actions/setup/action.yml` mtransitapps/commons#676 - commons: Build(deps): Bump com.google.firebase.crashlytics from 3.0.6 to 3.0.7 mtransitapps/commons#671 - commons: Support `MT.DELETE.<filename>` marker files in `shared-overwrite` for file deletion mtransitapps/commons#675 - commons: Build(deps): Bump the gms group with 2 updates mtransitapps/commons#669 - commons: Build(deps): Bump org.xerial:sqlite-jdbc from 3.51.3.0 to 3.53.0.0 mtransitapps/commons#672 - commons: Build(deps): Bump com.google.guava:guava from 33.5.0-android to 33.6.0-android mtransitapps/commons#673 - commons: Build(deps): Bump android-gradlePlugin from 9.1.0 to 9.1.1 mtransitapps/commons#670 - commons: Build(deps): Bump org.jetbrains.kotlinx:kotlinx-serialization-bom from 1.10.0 to 1.11.0 in the kotlin-ksp-compose group mtransitapps/commons#668 - commons: Replace local `setup` composite action by shared repo mtransitapps/commons#667 - commons: Rename `check_data_outdated.sh` → `check_data_not_outdated.sh`, keep wrapper for backward compat mtransitapps/commons#663 - commons: Circle CI: only build `master` branch mtransitapps/commons#665 - commons: Modify timeout settings for apt dependencies installation mtransitapps/commons#662 - commons-android: GTFS-RT > Trip Updates > Read from source > use `header.timestamp` mtransitapps/commons-android#125 - commons-android: TimeZone != TimeZoneId mtransitapps/commons-android#124 - commons-android: POI status > readFromSource Instant ext - commons-android: R8 full mode mtransitapps/commons-android#120 - commons-android: Update SSL cert mtransitapps/commons-android#122 - commons-android: Fix `debug` build != debuggable builds mtransitapps/commons-android#121 - commons-android: Keep PNGs in repo to avoid running image gen during all CI mtransitapps/commons-android#119 - commons-android: clean - commons-android: GTFS-RT > static<>real-time matching (out-of-sync `trip_id`...) & tests mtransitapps/commons-android#117 - commons-java: Strings cleaner > fix stop name length (default zoom) mtransitapps/commons-java#32 - commons-java: Add 'gtfs-prod' to the list of sources mtransitapps/commons-java#31 - parser: TimeZone != TimeZoneId mtransitapps/parser#61 - parser: Fix route provide missing color Java<>JSON config mess mtransitapps/parser#60