Skip to content
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

Expand expression evaluation tests to also test DDC module format + canary #2465

Merged
merged 1 commit into from
Aug 1, 2024

Conversation

srujzs
Copy link
Contributor

@srujzs srujzs commented Jul 29, 2024

DDC hot reload requires a new module format. This is represented by the intersection of the DDC module format and canary. In order to make sure the new format is tested, this changes replicates any existing expression evaluation tests to use the DDC module format and canary.

Copy link
Contributor

@nshahan nshahan left a comment

Choose a reason for hiding this comment

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

Thanks for all of the investigations we talked through!

@srujzs srujzs merged commit 3354ea5 into dart-lang:main Aug 1, 2024
50 of 52 checks passed
@srujzs
Copy link
Contributor Author

srujzs commented Aug 1, 2024

It looks like the recurring failure is test/variable_scope_test.dart but that seems to be a failure on main already looking at other runs, so I'm submitting.

Copy link
Collaborator

@bkonyi bkonyi left a comment

Choose a reason for hiding this comment

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

Sorry, not sure how I missed this! LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants