Support use-flutter option in groundskeeper workflow - #444
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the CHANGELOG.md for the firehose package to document the use of the Flutter dev channel in the groundskeeper workflow to support Flutter packages. There are no review comments, so I have no feedback to provide.
PR HealthUnused Dependencies ✔️
For details on how to fix these, see dependency_validator. This check can be disabled by tagging the PR with Breaking changes ✔️
This check can be disabled by tagging the PR with API leaks ✔️The following packages contain symbols visible in the public API, but not exported by the library. Export these symbols or remove them from your publicly visible API.
This check can be disabled by tagging the PR with Changelog Entry ✔️
Changes to files need to be accounted for in their respective changelogs. This check can be disabled by tagging the PR with License Headers ✔️
All source files should start with a license header. This check can be disabled by tagging the PR with Coverage ✔️
This check for test coverage is informational (issues shown here will not fail the PR). This check can be disabled by tagging the PR with |
Adds a
use-flutteroption (defaulting tofalse) togroundskeeper.yml.When
use-flutteristrue:subosito/flutter-actionwith the specified channel (sdk, defaulting todev) so that Flutter and Dart are available on PATH for Flutter packages (such ascronet_httporok_http).When
use-flutterisfalse(default):dart-lang/setup-dartwith the specified SDK version / channel (defaulting todev), keeping setup fast and lightweight for pure Dart packages.Addresses feedback from dart-lang/http#1959.