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

Revert "[compiler][eslint] remove compilationMode override; report bailouts on first line" #30792

Merged
merged 1 commit into from
Aug 22, 2024

Conversation

poteto
Copy link
Member

@poteto poteto commented Aug 22, 2024

Stack from ghstack (oldest at bottom):

This reverts commit b34b750.

This hack doesn't play well internally so I'm reverting this for now
(but keeping the compilationMode override). I'll audit the locations we
report later and try to make them more accurate so we won't need this
workaround.

[ghstack-poisoned]
Copy link

vercel bot commented Aug 22, 2024

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 Aug 22, 2024 7:02pm

poteto added a commit that referenced this pull request Aug 22, 2024
…ilouts on first line"

This reverts commit b34b750.

This hack doesn't play well internally so I'm reverting this for now
(but keeping the compilationMode override). I'll audit the locations we
report later and try to make them more accurate so we won't need this
workaround.

ghstack-source-id: b6be29c11d5e2eae3004ee74cba1b080894c963a
Pull Request resolved: #30792
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Aug 22, 2024
@poteto poteto merged commit 1fb14fd into gh/poteto/4/base Aug 22, 2024
19 checks passed
poteto added a commit that referenced this pull request Aug 22, 2024
…ilouts on first line"

This reverts commit b34b750.

This hack doesn't play well internally so I'm reverting this for now
(but keeping the compilationMode override). I'll audit the locations we
report later and try to make them more accurate so we won't need this
workaround.

ghstack-source-id: b6be29c11d5e2eae3004ee74cba1b080894c963a
Pull Request resolved: #30792
@poteto poteto deleted the gh/poteto/4/head branch August 22, 2024 19:04
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.

3 participants