-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
ESLint: Change jsdoc/check-line-alignment
from warn
to error
#47878
Conversation
Size Change: 0 B Total Size: 1.32 MB ℹ️ View Unchanged
|
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.
Thank you, @tyxla!
Flaky tests detected in 0ea771d. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/4124593300
|
I'm still debating with myself whether it should be only for this repository to start. On the other hand, folks can always disable this rule if that feels too much 🤔 |
I'm happy to move it to this project only if you feel strongly about it. I just think it can be helpful to everyone since it's been warning us for a while and I believe this isn't the kind of rule that should provide a warning. |
My only concern was the developer experience for folks who will start seeing errors after upgrading to the latest version, but since they see the warning already then maybe it isn't a big deal and aligns with how stricter rules are rolled out in other projects. Finally, it's a fixable rule, so if someone is using ESLint with this config, it's trivial to move forward. I think we are good 🚢 |
What?
This PR changes the
jsdoc/check-line-alignment
ESLint rule toerror
instead ofwarn
.Why?
We've recently been fixing a bunch of these warnings, so as suggested by @gziolo here and here it might be a good idea to promote it to
error
.How?
We're increasing the severity of the rule
jsdoc/check-line-alignment
fromwarn
toerror
.Testing Instructions
Verify
npm run lint:js
still works well and yields just 16 warnings.Testing Instructions for Keyboard
None
Screenshots or screencast
None