Skip to content

Conversation

@anonrig
Copy link
Member

@anonrig anonrig commented Dec 26, 2025

Another web-platform tests requirements. I've put it behind pedanticWpt flag...

@anonrig anonrig requested a review from jasnell December 26, 2025 21:47
@anonrig anonrig requested review from a team as code owners December 26, 2025 21:47
@anonrig anonrig force-pushed the yagiz/fix-desiredSize branch from a93109b to 8e5dd2e Compare December 29, 2025 15:28
@anonrig anonrig requested a review from jasnell December 29, 2025 15:29
@anonrig anonrig enabled auto-merge December 29, 2025 16:11
Copy link
Contributor

@guybedford guybedford left a comment

Choose a reason for hiding this comment

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

Conversely the to the previous PR, are we sure this will be breaking? Pedantic WPT not having a compat date is unfortunate for not auto-enabling?

@anonrig
Copy link
Member Author

anonrig commented Dec 29, 2025

Conversely the to the previous PR, are we sure this will be breaking? Pedantic WPT not having a compat date is unfortunate for not auto-enabling?

I'm not following @guybedford if (impl.flags.pedanticWpt && isErroring()) { This is already behind a pedantic wpt flag...

@anonrig anonrig requested a review from guybedford December 29, 2025 20:09
@jasnell
Copy link
Collaborator

jasnell commented Dec 29, 2025

This one is unlikely to actually be breaking but putting it behind the pedanticWpt flag also doesn't hurt. Should be ok either way.

@anonrig anonrig merged commit 33c030d into main Dec 29, 2025
32 of 33 checks passed
@anonrig anonrig deleted the yagiz/fix-desiredSize branch December 29, 2025 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants