Try: eslint-plugin-react-compiler
#61788
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
tyxla
added
[Type] Code Quality
|
Size Change: 0 B Total Size: 1.82 MB ℹ️ View Unchanged
|
|
It will trigger an ESLint error everywhere where we ignore a Seems like it will try hard to let us follow all the rules, which can create quite some additional work 😐 |
|
I kinda like the new stricter rules, but I agree it would take some time to bring the codebase into shape. P.S. I'll push fixes for some low-hanging errors separately. |
I'd love to help with those too. Please ping me for reviews! |
|
@tyxla, should we start a list of false positive errors that linter generates and then report those upstream? Example
|
Yep, definitely. The top-level await could also be one to report (currently not occurring because I've ignored the files where it happens). |
Mamaduka
reviewed
May 20, 2024
tyxla
force-pushed
the
try/eslint-plugin-react-compiler
branch
from
9a2d9a4
to
5921e1e
Compare
gziolo
reviewed
May 21, 2024
|
Flaky tests detected in 87dba34. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/11761860773
|
|
I noticed that new lint rules do not like the hook pattern used by the command API: Example: useCommandLoader( {
name: 'core/edit-site/reset-global-styles',
hook: useGlobalStylesResetCommands,
} ); |
The solution to that will most likely be just renaming them to not follow the hook naming convention. After all, they are technically not hooks since they're violating the rules of hooks and React can't recognize them as hooks through static code analysis: |
This was referenced May 23, 2024
tyxla
force-pushed
the
try/eslint-plugin-react-compiler
branch
from
54f002a
to
6bd3449
Compare
|
It looks like, after the latest update, the plugin complains about setting the |
Yes, looks like it's a known bug: facebook/react#29196 |
|
I'm talking about different case that's is supported in React and ref is coming from local I'm not able to reproduce it on React Compiler Playground though. Screenshot
|
|
Perhaps I'm misunderstanding, but isn't that precisely the bug reported in the issue I linked above? |
This was referenced Oct 26, 2024
|
I noticed that with React Compiler all |
Thanks @swissspidy. This issue belongs to the React Compiler itself, which you've rightfully also reported there. |
tyxla
force-pushed
the
try/eslint-plugin-react-compiler
branch
from
9e4d574
to
bb24372
Compare
|
⬇️ Down to 25 errors now. |
Most of those are related to Commands APIs hook style registration. |
Since I'd like to avoid API changes, I've prepared a workaround for this in #66787. Gets the errors down from 24 to 8 🥳 |
|
When I introduced the hook API, I didn't see any other alternative. That said, it's clear (for multiple reasons) that it's not a perfect API. I would love if we could think of a better one in the future. |
I totally agree that a new API is the best future solution. For now, though, I'd prefer to keep the changes contained. |
tyxla
force-pushed
the
try/eslint-plugin-react-compiler
branch
2 times, most recently
from
715790b
to
87a4312
Compare
This was referenced Nov 7, 2024
Co-authored-by: Pascal Birchler <pascalb@google.com>
tyxla
force-pushed
the
try/eslint-plugin-react-compiler
branch
from
87a4312
to
87dba34
Compare
|
⬇️ Only 2 errors left to go: 
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What?
This PR experiments with introducing the
eslint-plugin-react-compilerESLint plugin to the repo.The plugin is still experimental, so don't merge this just yet.
We're experimenting with adopting React Compiler through the Babel plugin in #66361.
Why?
It's recommended that this ESLint plugin is used to help improve the quality of your codebase.
How?
The
eslint-plugin-react-compilerplugin can be used without the React Compiler itself. We're installing it and enabling it. Also ignoring a few files that include top-levelawaitwhich seems to break the linter right now.Testing Instructions
Observe the new ESLint errors.
Testing Instructions for Keyboard
None
Screenshots or screencast
None