[go: nahoru, domu]

Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

[ci] Cirrus pre-alignment with flutter/packages, part 1 #6569

Merged
merged 13 commits into from
Oct 19, 2022

Conversation

stuartmorgan
Copy link
Contributor
@stuartmorgan stuartmorgan commented Oct 12, 2022

Minor changes in conjunction with flutter/packages#2706 to align the configurations.

  • Combines some very cheap steps that don't need to be separate, to reduce machine/setup overhead.
  • Minor naming/comment/flag adjustments
  • Cleans up some desktop enabling steps that are no longer necessary.
  • Use .ci/scripts/prepare_tool.sh, which had been set up but we hadn't switched Cirrus over to in this repo.

federated_safety_script:
# This check is only meaningful for PRs, as it validates changes
# rather than state.
- if [[ $CIRRUS_PR == "" ]]; then
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was cleaner as an only_if rather than a conditional no-op, but that only operates at the task level, and the overhead of the task takes much longer than the step itself so keeping it separate just didn't make sense.

Copy link
Contributor
@bparrishMines bparrishMines left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with some nits

.cirrus.yml Show resolved Hide resolved
.cirrus.yml Show resolved Hide resolved
.cirrus.yml Show resolved Hide resolved
@stuartmorgan stuartmorgan added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 19, 2022
@auto-submit auto-submit bot merged commit aeab040 into flutter:main Oct 19, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 20, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 20, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 20, 2022
adam-harwood pushed a commit to adam-harwood/flutter_plugins that referenced this pull request Nov 3, 2022
mauricioluz pushed a commit to mauricioluz/plugins that referenced this pull request Jan 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
autosubmit Merge PR when tree becomes green via auto submit App
Projects
None yet
2 participants