-
-
Notifications
You must be signed in to change notification settings - Fork 504
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
chore: promote rules #532
chore: promote rules #532
Conversation
Parser conformance results onjs/262
jsx/babel
symbols/microsoft
ts/babel
ts/microsoft
|
b37401a
to
1638fd0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
I think you forgot to promote noGlobalIsFinite to suspicious.
I could also rename noExcessiveComplexity
to noExcessiveCognitiveComplexity
.
Also we could promote and recommend noDuplicatePrivateClassMembers
and noSuperWithoutExtends
syntax rules?
These aren't lint rules, they are syntax rules. Users aren't able to configure them. They always run because they are meant to catch syntax errors or runtime errors. |
Is it normal the recommend flag is set to false and they are placed in a |
No, it seems that the script didn't work. Let me fix that, good catch |
f2b5495
to
8d945f7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you forgot to promote noGlobalIsFinite to suspicious.
I could also rename noExcessiveComplexity
to noExcessiveCognitiveComplexity
.
Also, a changelog entry is missing :)
Any suggestion/help? I don't know what's the expected changelog line |
We could use the same template that the previous version, creating a |
Done! |
crates/biome_js_analyze/src/analyzers/complexity/no_for_each.rs
Outdated
Show resolved
Hide resolved
4fbbe70
to
2f4c7e3
Compare
2f4c7e3
to
b6e9d61
Compare
b6e9d61
to
be64ce5
Compare
Summary
Closes #410
Test Plan
CI should pass