Skip to content

Conversation

@poteto
Copy link
Member

@poteto poteto commented Jan 7, 2025

The playground's compilation mode is currently set to 'all' along with reporting all errors.

This tends to be misleading since people usually expect a 1:1 match between how the playground works with what the compiler does in their codebase, eg reactwg/react-compiler#51.

The playground's compilation mode is currently set to 'all' along with reporting all errors.

This tends to be misleading since people usually expect a 1:1 match between how the playground works with what the compiler does in their codebase, eg reactwg/react-compiler#51.
@vercel
Copy link

vercel bot commented Jan 7, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
react-compiler-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 7, 2025 4:49pm

@facebook-github-bot facebook-github-bot added the React Core Team Opened by a member of the React Core Team label Jan 7, 2025
@poteto poteto merged commit 6efbc08 into main Jan 7, 2025
21 checks passed
@poteto poteto deleted the pr32009 branch January 7, 2025 16:53
poteto added a commit that referenced this pull request Jan 9, 2025
I had forgotten that our default error reporting threshold was `none` due to the fact that build pipelines should not throw errors. This resets it back to throwing on all errors which mostly is the same as the eslint plugin.

Closes #32014.
poteto added a commit that referenced this pull request Jan 9, 2025
I had forgotten that our default error reporting threshold was `none` due to the fact that build pipelines should not throw errors. This resets it back to throwing on all errors which mostly is the same as the eslint plugin.

Closes #32014.
poteto added a commit that referenced this pull request Jan 9, 2025
I had forgotten that our default error reporting threshold was `none`
due to the fact that build pipelines should not throw errors. This
resets it back to throwing on all errors which mostly is the same as the
eslint plugin.

Closes #32014.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants