[tool] Add config options needed by core-packages#11784
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces min_dart as an alternative to min_flutter in the repository tool configuration, updating the validation logic to check Dart SDK constraints accordingly. It also makes the .ci.yaml file optional during validation when batched releases are not used. Review feedback suggests handling potential FormatException errors when parsing version strings and correcting a configuration mismatch in the newly added tests where isFlutter: true was incorrectly set for non-Flutter package test cases.
| }) { | ||
| final errors = <String>[]; | ||
| final File ciYamlFile = repoRoot.childFile('.ci.yaml'); | ||
| if (!ciYamlFile.existsSync()) { |
There was a problem hiding this comment.
why do we bar batch release based on .ci.yaml? IIRC, batch release infra are mostly based on GA and didn't rely on ci.yaml. Or is this a indirect way to disable the batch release for core-package?
There was a problem hiding this comment.
Because the next non-new line in this file unconditionally reads ciYamlFile, and that will explode if, as is currently the case in core-packages, there is no such file.
Rather than speculatively implement an alternate codepath that does the same validation for a non-LUCI-based configuration that we've never built and may never need, I'm just describing the actual state of the tooling.
There was a problem hiding this comment.
oh ok, I think we can just return without the print error then.
The reason behind this check for batch release is the we have to add a enable_branch in ci.yaml to make sure the luci ci runs.
If it doesn't exist, then we don't need to worry about having the enable_branch check.
There was a problem hiding this comment.
Why wouldn't the same basic requirement apply to a repo using GA to run CI, given that GA also has branch restrictions in configurations?
There was a problem hiding this comment.
yeah the same will apply if we use GA to run ci, but this helper method is specific for ci.yaml.
There was a problem hiding this comment.
oh I see the reasoning, so until we implement the GA restriction, batch release is not supported. yeah I think that makes sense too.
stuartmorgan-g
added
the
autosubmit
This fixes everything that `validate` failed on locally when running flutter/packages#11784 locally against the repo. This is in preparation for enabling the repo tooling in this repo. - Adds dependabot coverage for example apps (although it's currently not going to actually do anything; it depends whether we ever add Flutter examples for non-Flutter-specific packages) - Adds autolabeling - Adds the tool config - Fixes repo-level README link mistakes - Requires Dart 3.10 (N-2) - Adds languages to code blocks in the README, and temporarily exempts the README from excerpts Tooling will actually be enabled in a future PR, once the tooling updates are published to pub.dev.
…r#187174) flutter/packages@fc795e5...4b424d7 2026-05-27 engine-flutter-autoroll@skia.org Roll Flutter from f3a4b98 to c8f2f16 (29 revisions) (flutter/packages#11795) 2026-05-27 stuartmorgan@google.com [google_maps_flutter] Remove duplicated privacy manifest entries (flutter/packages#11791) 2026-05-27 stuartmorgan@google.com [tool] Add config options needed by core-packages (flutter/packages#11784) 2026-05-26 kienhantrung@gmail.com [go_router] Allow users to specify onExit as optional (flutter/packages#11150) 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
Adds some optional behaviors needed to use the tool in flutter/core-packages: