Skip to content

[ci] Continue .cirrus.yml alignment with flutter/plugins #2832

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Nov 21, 2022

Conversation

stuartmorgan-g
Copy link
Contributor

Adds various tasks/checks that are currently in flutter/plugins but not flutter/packages to .cirrus.yaml. Most of these are currently harmless no-ops since they skip any packages without parts relevant to the check. Two substantive changes:

  • Enables Gradle dependabot to manage all of the example apps, as we have in flutter/plugins; this will help avoid the gradle versions in examples getting more and more outdated over time.
    • As part of this, removed the android/ directory from the pigeon Obj-C e2e test directory, which was presumably just created by default and left there. (I expect to remove the whole e2e directory once the new Pigeon integration tests are in place, but this is the minimal change for now.)
  • Moves Linux platform tests to the light workload group, which is where they are in flutter/plugins (since they don't actually need heavy workload machines).

Part of flutter/flutter#113764

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/packages repo does use dart format.)
  • I signed the CLA.
  • The title of the PR starts with the name of the package surrounded by square brackets, e.g. [shared_preferences]
  • I listed at least one issue that this PR fixes in the description above.
  • I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
  • I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

@gaaclarke
Copy link
Member

Ahh didn't see the ci failures. LGTM modulo them obviously. Ping me if a significant change is required to make them work and I can look again.

@stuartmorgan-g
Copy link
Contributor Author

Apparently Pigeon tests were generating the Java output for the ObjC e2e test 🤷🏻 I removed that too since it was unused.

@stuartmorgan-g stuartmorgan-g added the override: no versioning needed Override the check requiring version bumps for most changes label Nov 21, 2022
@stuartmorgan-g
Copy link
Contributor Author

Pigeon change is dev-only; overriding checks for it.

@stuartmorgan-g stuartmorgan-g added override: no changelog needed Override the check requiring CHANGELOG updates for most changes autosubmit Merge PR when tree becomes green via auto submit App labels Nov 21, 2022
@auto-submit auto-submit bot merged commit a683584 into flutter:main Nov 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autosubmit Merge PR when tree becomes green via auto submit App override: no changelog needed Override the check requiring CHANGELOG updates for most changes override: no versioning needed Override the check requiring version bumps for most changes p: pigeon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants