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

Only set dart_include_wasm_opt when we are actually building the dart sdk.#41175

Merged
auto-submit[bot] merged 1 commit intoflutter:mainfrom
eyebrowsoffire:dart_wasm_opt_warning
Apr 14, 2023
Merged

Only set dart_include_wasm_opt when we are actually building the dart sdk.#41175
auto-submit[bot] merged 1 commit intoflutter:mainfrom
eyebrowsoffire:dart_wasm_opt_warning

Conversation

@eyebrowsoffire
Copy link
Contributor

This fixes flutter/flutter#123629

We were getting a warning when specifying this and not building the dart SDK.

@eyebrowsoffire eyebrowsoffire added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 13, 2023
@flutter-dashboard
Copy link

Golden file changes have been found for this pull request. Click here to view and triage (e.g. because this is an intentional change).

If you are still iterating on this change and are not ready to resolve the images on the Flutter Gold dashboard, consider marking this PR as a draft pull request above. You will still be able to view image results on the dashboard, commenting will be silenced, and the check will not try to resolve itself until marked ready for review.

Changes reported for pull request #41175 at sha d031d41

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 will affect goldens

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GN warning about dart_include_wasm_opt in some configs

2 participants