Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@zanderso
Copy link
Member

@zanderso zanderso commented Oct 2, 2024

This is my bad for not having this documented better/at-all, but since this build does not produce artifacts, it should not go under a builder that is marked as release_build: "true" in the .ci.yaml. This PR moves the build to linux_unopt.json.

@zanderso zanderso requested review from aam and matanlurey October 2, 2024 22:29
@zanderso zanderso added autosubmit Merge PR when tree becomes green via auto submit App and removed autosubmit Merge PR when tree becomes green via auto submit App labels Oct 2, 2024
@zanderso zanderso force-pushed the move-no-prebuilt-dart-build branch from 1155e77 to fbf6a91 Compare October 2, 2024 22:46
@zanderso
Copy link
Member Author

zanderso commented Oct 2, 2024

Added some comments to the json files. Unfortunately, there's a little more cleanup to do before a test could enforce this: There are still some tests running where they shouldn't be, and some builds without artifacts sections have their build products harvested in global generators/global packaging sections.

@zanderso zanderso added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 2, 2024
Copy link
Member

@aam aam left a comment

Choose a reason for hiding this comment

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

thanks, Zach, good to know!

@auto-submit auto-submit bot merged commit bd44b58 into flutter:main Oct 2, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 3, 2024
@zanderso zanderso deleted the move-no-prebuilt-dart-build branch October 8, 2024 23:46
nick9822 pushed a commit to nick9822/flutter that referenced this pull request Dec 18, 2024
This is my bad for not having this documented better/at-all, but since this build does not produce artifacts, it should not go under a builder that is marked as `release_build: "true"` in the .ci.yaml. This PR moves the build to `linux_unopt.json`.
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

Development

Successfully merging this pull request may close these issues.

3 participants