Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
[ci] Always run all format steps
The `format` task runs several independent steps, which are grouped
together only to avoid unnecessary overhead from running each
inexpensive check on its own VM. Since they are unrelated and
inexpensive, we should always run all of them so that we can get all
failures at once, rather than only seeing issues from the first failed
step, and needing to potentially run through the bot three or more
times to get all failures.
  • Loading branch information
stuartmorgan-g committed Oct 12, 2021
commit 01a94af911af1b628da17216d91e23da085409b0
11 changes: 6 additions & 5 deletions .cirrus.yml
Original file line number Diff line number Diff line change
Expand Up @@ -69,8 +69,8 @@ task:
- dart pub run test
- name: publishable
env:
# TODO (mvanbeusekom): Temporary override to "stable" because of failure on "master".
# Remove override once https://github.com/dart-lang/pub/issues/3152 is resolved.
# TODO (mvanbeusekom): Temporary override to "stable" because of failure on "master".
# Remove override once https://github.com/dart-lang/pub/issues/3152 is resolved.
CHANNEL: stable
CHANGE_DESC: "$TMPDIR/change-description.txt"
version_check_script:
Expand All @@ -88,9 +88,10 @@ task:
- fi
publish_check_script: ./script/tool_runner.sh publish-check
- name: format
format_script: ./script/tool_runner.sh format --fail-on-change
pubspec_script: ./script/tool_runner.sh pubspec-check
license_script: dart $PLUGIN_TOOL license-check
always:
format_script: ./script/tool_runner.sh format --fail-on-change
pubspec_script: ./script/tool_runner.sh pubspec-check
license_script: dart $PLUGIN_TOOL license-check
- name: federated_safety
# This check is only meaningful for PRs, as it validates changes
# rather than state.
Expand Down