Conversation
… skip for data download and app release Co-authored-by: mmathieum <177998+mmathieum@users.noreply.github.com>
|
TODO:
|
mt-sync-code-data workflow on PRs with automatic skip of data download and app release
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request successfully enables the mt-sync-code-data workflow to run on pull requests, correctly skipping expensive operations like data downloads and app releases. The introduction of a composite action actions/setup is a great improvement for modularity and maintainability. I've found one issue with the logic for scheduled runs which could prevent data from being downloaded. My review includes a suggestion to fix this.
|
commons/.github/workflows/mt-build.yml Line 14 in fb9cfbb Itself calling |
…parser': - commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons: Convert PR references to full URLs in submodule sync commits mtransitapps/commons#533 - commons: Fix archive.sh: remove old archives when current archive exists mtransitapps/commons#536 - commons: prepare_only.sh (archive_selection.sh & unzip_gtfs.sh) > remove old gtfs next data if URL not availabla anymore - commons: Build(deps): Bump the maps group with 3 updates mtransitapps/commons#530 - commons: Build(deps): Bump com.google.ads.mediation:vungle from 7.6.3.0 to 7.6.3.1 in the ads group mtransitapps/commons#529 - commons: Build(deps): Bump com.google.devtools.ksp from 2.3.4 to 2.3.5 in the kotlin-ksp-compose group mtransitapps/commons#528 - commons: Compat w/ Winnipeg transit provider - commons: Compat w/ Greater Sudbury MyBus API - commons: Compat w/ Regina Transit provider - commons: Compat w/ QuebecRTC provider - commons: Compat w/ STM.info providers - commons: Compat w/ network security config - commons: Compat w/ GRT provider - commons: Compat w/ CaEdmontonProvider - commons: Compat w/ Strategic Mapping provider - commons: AndroidManifest.xml gen > EXPORT_VEHICLE_LOCATION_PROVIDER FF (OFF) - commons: AndroidManifest.xml gen > compat w/ NextBus #VehicleLocationProvider - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-android: Change DEBUG constant to use environment variable mtransitapps/commons-android#52 - commons-java: cleanup - commons-java: Change DEBUG constant to use environment variable mtransitapps/commons-java#15 - parser: Reader > fix trip_ids file name - parser: Revert "MSchedule > clear head-sign instead of setting empty string" - parser: MSchedule > clear head-sign instead of setting empty string - parser: Current schedule > only look backward for single-day service if already present in current schedule... - parser: Change DEBUG constant to use environment variable mtransitapps/parser#29 - parser: Avoid data change > reading service dates > compat w/ flatten FF - parser: Export trip IDs [ints], service IDs, [schedule] strings FFs > delete old files if FF is ON->OFF - parser: Better clean mergedIds (+trip Ids)
- commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-java: cleanup
- commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-java: cleanup
…parser': - commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons: Convert PR references to full URLs in submodule sync commits mtransitapps/commons#533 - commons: Fix archive.sh: remove old archives when current archive exists mtransitapps/commons#536 - commons: prepare_only.sh (archive_selection.sh & unzip_gtfs.sh) > remove old gtfs next data if URL not availabla anymore - commons: Build(deps): Bump the maps group with 3 updates mtransitapps/commons#530 - commons: Build(deps): Bump com.google.ads.mediation:vungle from 7.6.3.0 to 7.6.3.1 in the ads group mtransitapps/commons#529 - commons: Build(deps): Bump com.google.devtools.ksp from 2.3.4 to 2.3.5 in the kotlin-ksp-compose group mtransitapps/commons#528 - commons: Compat w/ Winnipeg transit provider - commons: Compat w/ Greater Sudbury MyBus API - commons: Compat w/ Regina Transit provider - commons: Compat w/ QuebecRTC provider - commons: Compat w/ STM.info providers - commons: Compat w/ network security config - commons: Compat w/ GRT provider - commons: Compat w/ CaEdmontonProvider - commons: Compat w/ Strategic Mapping provider - commons: AndroidManifest.xml gen > EXPORT_VEHICLE_LOCATION_PROVIDER FF (OFF) - commons: AndroidManifest.xml gen > compat w/ NextBus #VehicleLocationProvider - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-android: Change DEBUG constant to use environment variable mtransitapps/commons-android#52 - commons-java: cleanup - commons-java: Change DEBUG constant to use environment variable mtransitapps/commons-java#15 - parser: Reader > fix trip_ids file name - parser: Revert "MSchedule > clear head-sign instead of setting empty string" - parser: MSchedule > clear head-sign instead of setting empty string - parser: Current schedule > only look backward for single-day service if already present in current schedule... - parser: Change DEBUG constant to use environment variable mtransitapps/parser#29 - parser: Avoid data change > reading service dates > compat w/ flatten FF - parser: Export trip IDs [ints], service IDs, [schedule] strings FFs > delete old files if FF is ON->OFF - parser: Better clean mergedIds (+trip Ids)
…parser': - commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons: Convert PR references to full URLs in submodule sync commits mtransitapps/commons#533 - commons: Fix archive.sh: remove old archives when current archive exists mtransitapps/commons#536 - commons: prepare_only.sh (archive_selection.sh & unzip_gtfs.sh) > remove old gtfs next data if URL not availabla anymore - commons: Build(deps): Bump the maps group with 3 updates mtransitapps/commons#530 - commons: Build(deps): Bump com.google.ads.mediation:vungle from 7.6.3.0 to 7.6.3.1 in the ads group mtransitapps/commons#529 - commons: Build(deps): Bump com.google.devtools.ksp from 2.3.4 to 2.3.5 in the kotlin-ksp-compose group mtransitapps/commons#528 - commons: Compat w/ Winnipeg transit provider - commons: Compat w/ Greater Sudbury MyBus API - commons: Compat w/ Regina Transit provider - commons: Compat w/ QuebecRTC provider - commons: Compat w/ STM.info providers - commons: Compat w/ network security config - commons: Compat w/ GRT provider - commons: Compat w/ CaEdmontonProvider - commons: Compat w/ Strategic Mapping provider - commons: AndroidManifest.xml gen > EXPORT_VEHICLE_LOCATION_PROVIDER FF (OFF) - commons: AndroidManifest.xml gen > compat w/ NextBus #VehicleLocationProvider - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-android: Change DEBUG constant to use environment variable mtransitapps/commons-android#52 - commons-java: cleanup - commons-java: Change DEBUG constant to use environment variable mtransitapps/commons-java#15 - parser: Reader > fix trip_ids file name - parser: Revert "MSchedule > clear head-sign instead of setting empty string" - parser: MSchedule > clear head-sign instead of setting empty string - parser: Current schedule > only look backward for single-day service if already present in current schedule... - parser: Change DEBUG constant to use environment variable mtransitapps/parser#29 - parser: Avoid data change > reading service dates > compat w/ flatten FF - parser: Export trip IDs [ints], service IDs, [schedule] strings FFs > delete old files if FF is ON->OFF - parser: Better clean mergedIds (+trip Ids)
- commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-java: cleanup
- commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-java: cleanup
…parser': - commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons: Convert PR references to full URLs in submodule sync commits mtransitapps/commons#533 - commons: Fix archive.sh: remove old archives when current archive exists mtransitapps/commons#536 - commons: prepare_only.sh (archive_selection.sh & unzip_gtfs.sh) > remove old gtfs next data if URL not availabla anymore - commons: Build(deps): Bump the maps group with 3 updates mtransitapps/commons#530 - commons: Build(deps): Bump com.google.ads.mediation:vungle from 7.6.3.0 to 7.6.3.1 in the ads group mtransitapps/commons#529 - commons: Build(deps): Bump com.google.devtools.ksp from 2.3.4 to 2.3.5 in the kotlin-ksp-compose group mtransitapps/commons#528 - commons: Compat w/ Winnipeg transit provider - commons: Compat w/ Greater Sudbury MyBus API - commons: Compat w/ Regina Transit provider - commons: Compat w/ QuebecRTC provider - commons: Compat w/ STM.info providers - commons: Compat w/ network security config - commons: Compat w/ GRT provider - commons: Compat w/ CaEdmontonProvider - commons: Compat w/ Strategic Mapping provider - commons: AndroidManifest.xml gen > EXPORT_VEHICLE_LOCATION_PROVIDER FF (OFF) - commons: AndroidManifest.xml gen > compat w/ NextBus #VehicleLocationProvider - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-android: Change DEBUG constant to use environment variable mtransitapps/commons-android#52 - commons-java: cleanup - commons-java: Change DEBUG constant to use environment variable mtransitapps/commons-java#15 - parser: Reader > fix trip_ids file name - parser: Revert "MSchedule > clear head-sign instead of setting empty string" - parser: MSchedule > clear head-sign instead of setting empty string - parser: Current schedule > only look backward for single-day service if already present in current schedule... - parser: Change DEBUG constant to use environment variable mtransitapps/parser#29 - parser: Avoid data change > reading service dates > compat w/ flatten FF - parser: Export trip IDs [ints], service IDs, [schedule] strings FFs > delete old files if FF is ON->OFF - parser: Better clean mergedIds (+trip Ids)
- commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-java: cleanup
- commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons: Convert PR references to full URLs in submodule sync commits mtransitapps/commons#533 - commons: Fix archive.sh: remove old archives when current archive exists mtransitapps/commons#536 - commons: prepare_only.sh (archive_selection.sh & unzip_gtfs.sh) > remove old gtfs next data if URL not availabla anymore - commons: Build(deps): Bump the maps group with 3 updates mtransitapps/commons#530 - commons: Build(deps): Bump com.google.ads.mediation:vungle from 7.6.3.0 to 7.6.3.1 in the ads group mtransitapps/commons#529 - commons: Build(deps): Bump com.google.devtools.ksp from 2.3.4 to 2.3.5 in the kotlin-ksp-compose group mtransitapps/commons#528 - commons: Compat w/ Winnipeg transit provider - commons: Compat w/ Greater Sudbury MyBus API - commons: Compat w/ Regina Transit provider - commons: Compat w/ QuebecRTC provider - commons: Compat w/ STM.info providers - commons: Compat w/ network security config - commons: Compat w/ GRT provider - commons: Compat w/ CaEdmontonProvider - commons: Compat w/ Strategic Mapping provider - commons: AndroidManifest.xml gen > EXPORT_VEHICLE_LOCATION_PROVIDER FF (OFF) - commons: AndroidManifest.xml gen > compat w/ NextBus #VehicleLocationProvider - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-android: Change DEBUG constant to use environment variable mtransitapps/commons-android#52 - commons-java: cleanup - commons-java: Change DEBUG constant to use environment variable mtransitapps/commons-java#15
- commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-java: cleanup
- commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-java: cleanup
- commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-java: cleanup
- commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-java: cleanup
…parser': - commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons: Convert PR references to full URLs in submodule sync commits mtransitapps/commons#533 - commons: Fix archive.sh: remove old archives when current archive exists mtransitapps/commons#536 - commons: prepare_only.sh (archive_selection.sh & unzip_gtfs.sh) > remove old gtfs next data if URL not availabla anymore - commons: Build(deps): Bump the maps group with 3 updates mtransitapps/commons#530 - commons: Build(deps): Bump com.google.ads.mediation:vungle from 7.6.3.0 to 7.6.3.1 in the ads group mtransitapps/commons#529 - commons: Build(deps): Bump com.google.devtools.ksp from 2.3.4 to 2.3.5 in the kotlin-ksp-compose group mtransitapps/commons#528 - commons: Compat w/ Winnipeg transit provider - commons: Compat w/ Greater Sudbury MyBus API - commons: Compat w/ Regina Transit provider - commons: Compat w/ QuebecRTC provider - commons: Compat w/ STM.info providers - commons: Compat w/ network security config - commons: Compat w/ GRT provider - commons: Compat w/ CaEdmontonProvider - commons: Compat w/ Strategic Mapping provider - commons: AndroidManifest.xml gen > EXPORT_VEHICLE_LOCATION_PROVIDER FF (OFF) - commons: AndroidManifest.xml gen > compat w/ NextBus #VehicleLocationProvider - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-android: Change DEBUG constant to use environment variable mtransitapps/commons-android#52 - commons-java: cleanup - commons-java: Change DEBUG constant to use environment variable mtransitapps/commons-java#15 - parser: Reader > fix trip_ids file name - parser: Revert "MSchedule > clear head-sign instead of setting empty string" - parser: MSchedule > clear head-sign instead of setting empty string - parser: Current schedule > only look backward for single-day service if already present in current schedule... - parser: Change DEBUG constant to use environment variable mtransitapps/parser#29 - parser: Avoid data change > reading service dates > compat w/ flatten FF - parser: Export trip IDs [ints], service IDs, [schedule] strings FFs > delete old files if FF is ON->OFF - parser: Better clean mergedIds (+trip Ids)
…parser': - commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons: Convert PR references to full URLs in submodule sync commits mtransitapps/commons#533 - commons: Fix archive.sh: remove old archives when current archive exists mtransitapps/commons#536 - commons: prepare_only.sh (archive_selection.sh & unzip_gtfs.sh) > remove old gtfs next data if URL not availabla anymore - commons: Build(deps): Bump the maps group with 3 updates mtransitapps/commons#530 - commons: Build(deps): Bump com.google.ads.mediation:vungle from 7.6.3.0 to 7.6.3.1 in the ads group mtransitapps/commons#529 - commons: Build(deps): Bump com.google.devtools.ksp from 2.3.4 to 2.3.5 in the kotlin-ksp-compose group mtransitapps/commons#528 - commons: Compat w/ Winnipeg transit provider - commons: Compat w/ Greater Sudbury MyBus API - commons: Compat w/ Regina Transit provider - commons: Compat w/ QuebecRTC provider - commons: Compat w/ STM.info providers - commons: Compat w/ network security config - commons: Compat w/ GRT provider - commons: Compat w/ CaEdmontonProvider - commons: Compat w/ Strategic Mapping provider - commons: AndroidManifest.xml gen > EXPORT_VEHICLE_LOCATION_PROVIDER FF (OFF) - commons: AndroidManifest.xml gen > compat w/ NextBus #VehicleLocationProvider - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-android: Change DEBUG constant to use environment variable mtransitapps/commons-android#52 - commons-java: cleanup - commons-java: Change DEBUG constant to use environment variable mtransitapps/commons-java#15 - parser: Reader > fix trip_ids file name - parser: Revert "MSchedule > clear head-sign instead of setting empty string" - parser: MSchedule > clear head-sign instead of setting empty string - parser: Current schedule > only look backward for single-day service if already present in current schedule... - parser: Change DEBUG constant to use environment variable mtransitapps/parser#29 - parser: Avoid data change > reading service dates > compat w/ flatten FF - parser: Export trip IDs [ints], service IDs, [schedule] strings FFs > delete old files if FF is ON->OFF - parser: Better clean mergedIds (+trip Ids)
…parser': - commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons: Convert PR references to full URLs in submodule sync commits mtransitapps/commons#533 - commons: Fix archive.sh: remove old archives when current archive exists mtransitapps/commons#536 - commons: prepare_only.sh (archive_selection.sh & unzip_gtfs.sh) > remove old gtfs next data if URL not availabla anymore - commons: Build(deps): Bump the maps group with 3 updates mtransitapps/commons#530 - commons: Build(deps): Bump com.google.ads.mediation:vungle from 7.6.3.0 to 7.6.3.1 in the ads group mtransitapps/commons#529 - commons: Build(deps): Bump com.google.devtools.ksp from 2.3.4 to 2.3.5 in the kotlin-ksp-compose group mtransitapps/commons#528 - commons: Compat w/ Winnipeg transit provider - commons: Compat w/ Greater Sudbury MyBus API - commons: Compat w/ Regina Transit provider - commons: Compat w/ QuebecRTC provider - commons: Compat w/ STM.info providers - commons: Compat w/ network security config - commons: Compat w/ GRT provider - commons: Compat w/ CaEdmontonProvider - commons: Compat w/ Strategic Mapping provider - commons: AndroidManifest.xml gen > EXPORT_VEHICLE_LOCATION_PROVIDER FF (OFF) - commons: AndroidManifest.xml gen > compat w/ NextBus #VehicleLocationProvider - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-android: Change DEBUG constant to use environment variable mtransitapps/commons-android#52 - commons-java: cleanup - commons-java: Change DEBUG constant to use environment variable mtransitapps/commons-java#15 - parser: Reader > fix trip_ids file name - parser: Revert "MSchedule > clear head-sign instead of setting empty string" - parser: MSchedule > clear head-sign instead of setting empty string - parser: Current schedule > only look backward for single-day service if already present in current schedule... - parser: Change DEBUG constant to use environment variable mtransitapps/parser#29 - parser: Avoid data change > reading service dates > compat w/ flatten FF - parser: Export trip IDs [ints], service IDs, [schedule] strings FFs > delete old files if FF is ON->OFF - parser: Better clean mergedIds (+trip Ids)
…parser': - commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons: Convert PR references to full URLs in submodule sync commits mtransitapps/commons#533 - commons: Fix archive.sh: remove old archives when current archive exists mtransitapps/commons#536 - commons: prepare_only.sh (archive_selection.sh & unzip_gtfs.sh) > remove old gtfs next data if URL not availabla anymore - commons: Build(deps): Bump the maps group with 3 updates mtransitapps/commons#530 - commons: Build(deps): Bump com.google.ads.mediation:vungle from 7.6.3.0 to 7.6.3.1 in the ads group mtransitapps/commons#529 - commons: Build(deps): Bump com.google.devtools.ksp from 2.3.4 to 2.3.5 in the kotlin-ksp-compose group mtransitapps/commons#528 - commons: Compat w/ Winnipeg transit provider - commons: Compat w/ Greater Sudbury MyBus API - commons: Compat w/ Regina Transit provider - commons: Compat w/ QuebecRTC provider - commons: Compat w/ STM.info providers - commons: Compat w/ network security config - commons: Compat w/ GRT provider - commons: Compat w/ CaEdmontonProvider - commons: Compat w/ Strategic Mapping provider - commons: AndroidManifest.xml gen > EXPORT_VEHICLE_LOCATION_PROVIDER FF (OFF) - commons: AndroidManifest.xml gen > compat w/ NextBus #VehicleLocationProvider - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-android: Change DEBUG constant to use environment variable mtransitapps/commons-android#52 - commons-java: cleanup - commons-java: Change DEBUG constant to use environment variable mtransitapps/commons-java#15 - parser: Reader > fix trip_ids file name - parser: Revert "MSchedule > clear head-sign instead of setting empty string" - parser: MSchedule > clear head-sign instead of setting empty string - parser: Current schedule > only look backward for single-day service if already present in current schedule... - parser: Change DEBUG constant to use environment variable mtransitapps/parser#29 - parser: Avoid data change > reading service dates > compat w/ flatten FF - parser: Export trip IDs [ints], service IDs, [schedule] strings FFs > delete old files if FF is ON->OFF - parser: Better clean mergedIds (+trip Ids)
- commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-java: cleanup
…parser': - commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons: Convert PR references to full URLs in submodule sync commits mtransitapps/commons#533 - commons: Fix archive.sh: remove old archives when current archive exists mtransitapps/commons#536 - commons: prepare_only.sh (archive_selection.sh & unzip_gtfs.sh) > remove old gtfs next data if URL not availabla anymore - commons: Build(deps): Bump the maps group with 3 updates mtransitapps/commons#530 - commons: Build(deps): Bump com.google.ads.mediation:vungle from 7.6.3.0 to 7.6.3.1 in the ads group mtransitapps/commons#529 - commons: Build(deps): Bump com.google.devtools.ksp from 2.3.4 to 2.3.5 in the kotlin-ksp-compose group mtransitapps/commons#528 - commons: Compat w/ Winnipeg transit provider - commons: Compat w/ Greater Sudbury MyBus API - commons: Compat w/ Regina Transit provider - commons: Compat w/ QuebecRTC provider - commons: Compat w/ STM.info providers - commons: Compat w/ network security config - commons: Compat w/ GRT provider - commons: Compat w/ CaEdmontonProvider - commons: Compat w/ Strategic Mapping provider - commons: AndroidManifest.xml gen > EXPORT_VEHICLE_LOCATION_PROVIDER FF (OFF) - commons: AndroidManifest.xml gen > compat w/ NextBus #VehicleLocationProvider - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-android: Change DEBUG constant to use environment variable mtransitapps/commons-android#52 - commons-java: cleanup - commons-java: Change DEBUG constant to use environment variable mtransitapps/commons-java#15 - parser: Reader > fix trip_ids file name - parser: Revert "MSchedule > clear head-sign instead of setting empty string" - parser: MSchedule > clear head-sign instead of setting empty string - parser: Current schedule > only look backward for single-day service if already present in current schedule... - parser: Change DEBUG constant to use environment variable mtransitapps/parser#29 - parser: Avoid data change > reading service dates > compat w/ flatten FF - parser: Export trip IDs [ints], service IDs, [schedule] strings FFs > delete old files if FF is ON->OFF - parser: Better clean mergedIds (+trip Ids)
- commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-java: cleanup
- commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-java: cleanup
…parser': - commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons: Convert PR references to full URLs in submodule sync commits mtransitapps/commons#533 - commons: Fix archive.sh: remove old archives when current archive exists mtransitapps/commons#536 - commons: prepare_only.sh (archive_selection.sh & unzip_gtfs.sh) > remove old gtfs next data if URL not availabla anymore - commons: Build(deps): Bump the maps group with 3 updates mtransitapps/commons#530 - commons: Build(deps): Bump com.google.ads.mediation:vungle from 7.6.3.0 to 7.6.3.1 in the ads group mtransitapps/commons#529 - commons: Build(deps): Bump com.google.devtools.ksp from 2.3.4 to 2.3.5 in the kotlin-ksp-compose group mtransitapps/commons#528 - commons: Compat w/ Winnipeg transit provider - commons: Compat w/ Greater Sudbury MyBus API - commons: Compat w/ Regina Transit provider - commons: Compat w/ QuebecRTC provider - commons: Compat w/ STM.info providers - commons: Compat w/ network security config - commons: Compat w/ GRT provider - commons: Compat w/ CaEdmontonProvider - commons: Compat w/ Strategic Mapping provider - commons: AndroidManifest.xml gen > EXPORT_VEHICLE_LOCATION_PROVIDER FF (OFF) - commons: AndroidManifest.xml gen > compat w/ NextBus #VehicleLocationProvider - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-android: Change DEBUG constant to use environment variable mtransitapps/commons-android#52 - commons-java: cleanup - commons-java: Change DEBUG constant to use environment variable mtransitapps/commons-java#15 - parser: Reader > fix trip_ids file name - parser: Revert "MSchedule > clear head-sign instead of setting empty string" - parser: MSchedule > clear head-sign instead of setting empty string - parser: Current schedule > only look backward for single-day service if already present in current schedule... - parser: Change DEBUG constant to use environment variable mtransitapps/parser#29 - parser: Avoid data change > reading service dates > compat w/ flatten FF - parser: Export trip IDs [ints], service IDs, [schedule] strings FFs > delete old files if FF is ON->OFF - parser: Better clean mergedIds (+trip Ids)
- commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-java: cleanup
- commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup
- commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-java: cleanup
- commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-java: cleanup
…parser': - commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons: Convert PR references to full URLs in submodule sync commits mtransitapps/commons#533 - commons: Fix archive.sh: remove old archives when current archive exists mtransitapps/commons#536 - commons: prepare_only.sh (archive_selection.sh & unzip_gtfs.sh) > remove old gtfs next data if URL not availabla anymore - commons: Build(deps): Bump the maps group with 3 updates mtransitapps/commons#530 - commons: Build(deps): Bump com.google.ads.mediation:vungle from 7.6.3.0 to 7.6.3.1 in the ads group mtransitapps/commons#529 - commons: Build(deps): Bump com.google.devtools.ksp from 2.3.4 to 2.3.5 in the kotlin-ksp-compose group mtransitapps/commons#528 - commons: Compat w/ Winnipeg transit provider - commons: Compat w/ Greater Sudbury MyBus API - commons: Compat w/ Regina Transit provider - commons: Compat w/ QuebecRTC provider - commons: Compat w/ STM.info providers - commons: Compat w/ network security config - commons: Compat w/ GRT provider - commons: Compat w/ CaEdmontonProvider - commons: Compat w/ Strategic Mapping provider - commons: AndroidManifest.xml gen > EXPORT_VEHICLE_LOCATION_PROVIDER FF (OFF) - commons: AndroidManifest.xml gen > compat w/ NextBus #VehicleLocationProvider - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-android: Change DEBUG constant to use environment variable mtransitapps/commons-android#52 - commons-java: cleanup - commons-java: Change DEBUG constant to use environment variable mtransitapps/commons-java#15 - parser: Reader > fix trip_ids file name - parser: Revert "MSchedule > clear head-sign instead of setting empty string" - parser: MSchedule > clear head-sign instead of setting empty string - parser: Current schedule > only look backward for single-day service if already present in current schedule... - parser: Change DEBUG constant to use environment variable mtransitapps/parser#29 - parser: Avoid data change > reading service dates > compat w/ flatten FF - parser: Export trip IDs [ints], service IDs, [schedule] strings FFs > delete old files if FF is ON->OFF - parser: Better clean mergedIds (+trip Ids)
- commons: Enable `mt-sync-code-data` workflow on PRs with automatic skip of data download and app release mtransitapps/commons#539 - commons: app_reinstall_downgrade_debug_latest.sh ignore - commons: Remove `setGitBranch` from `code_setup.sh` and `set_app_release_required.sh` mtransitapps/commons#542 - commons: Add script for reinstalling debug APK with version downgrade support mtransitapps/commons#526 - commons: Fix PR reference in commits do not support markdown links, just use URL mtransitapps/commons#541 - commons-android: BUG > map marker icons are greens (more visible w/ low DPIs) mtransitapps/commons-android#53 - commons-android: cleanup - commons-java: cleanup
Enables the workflow to run on pull requests while automatically skipping expensive operations (data downloads and app releases) that are unnecessary for PR validation.
Changes
pull_requesttrigger to workflowgithub.event_name != 'pull_request':PR runs perform code sync and validation (parse current/next, list changes) without downloading fresh data or building releases.
Original prompt
mt-sync-code-data.ymlshould also run on pulls request but automatically skip downloading new data and app release #538💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.