Skip to content

Add invalidation test for optional other inputs. #3990

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

Conversation

davidmorgan
Copy link
Contributor

For #3811.

This is now enough coverage to catch the bug in my pending/next PR :)

Optional generated files are not built in phase order, but only if needed by something non-optional at a later phase. So this exercises different codepaths.

Copy link

PR Health

Changelog Entry ✔️
Package Changed Files

Changes to files need to be accounted for in their respective changelogs.

@davidmorgan davidmorgan requested a review from jensjoha May 1, 2025 08:03
@davidmorgan davidmorgan merged commit 5558265 into dart-lang:master May 1, 2025
74 of 75 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants