Skip to content

Conversation

poteto
Copy link
Member

@poteto poteto commented Apr 24, 2025

poteto added 2 commits April 24, 2025 13:13
Projects with existing babel config files may confuse the LSP, so explictly opt out of looking them up.
Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:
@github-actions github-actions bot added the React Core Team Opened by a member of the React Core Team label Apr 24, 2025
poteto added a commit that referenced this pull request Apr 24, 2025
Projects with existing babel config files may confuse the LSP, so
explictly opt out of looking them up.
---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed
with [ReviewStack](https://reviewstack.dev/facebook/react/pull/33010).
* #33012
* #33011
* __->__ #33010
@poteto poteto merged commit 2af218a into main Apr 24, 2025
26 checks passed
poteto added a commit that referenced this pull request Apr 24, 2025
Previously the CompileSuccess event would emit first before CompileSkip,
so the lsp's codelens would incorrectly flag skipped components/hooks
(via 'use no memo') as being optimized.
---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed
with [ReviewStack](https://reviewstack.dev/facebook/react/pull/33012).
* __->__ #33012
* #33011
* #33010
github-actions bot pushed a commit that referenced this pull request Apr 24, 2025
Previously the CompileSuccess event would emit first before CompileSkip,
so the lsp's codelens would incorrectly flag skipped components/hooks
(via 'use no memo') as being optimized.
---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed
with [ReviewStack](https://reviewstack.dev/facebook/react/pull/33012).
* __->__ #33012
* #33011
* #33010

DiffTrain build for [9938f83](9938f83)
@poteto poteto deleted the pr33011 branch April 24, 2025 18:58
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.

2 participants