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: enable jsdoc/require-description #12365
Conversation
@@ -67,7 +67,7 @@ describe("RuleTester", () => { | |||
let ruleTester; | |||
|
|||
/** | |||
* @description A helper function to verify Node.js core error messages. | |||
* A helper function to verify Node.js core error messages. |
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.
The default configuration doesn't check for @description
tags. We can use descriptionStyle: "any"
to allow both styles, but since this is the only instance in our codebase, it seemed like a good idea to just go with the default behavior. Happy to change it if others disagree, though.
32a392d
to
6f367b0
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.
LGTM, thank you!
What is the purpose of this pull request? (put an "X" next to item)
[ ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofixing to a rule
[ ] Add a CLI option
[ ] Add something to the core
[x] Other, please explain:
What changes did you make? (Give an overview)
Following up on #12336 (review), this PR enables the
jsdoc/require-description
rule, usingeslint-disable-*
comments for the 30 instances where a description is missing. As @mysticatea suggested, this will ensure we don't add any new JSDoc blocks that are missing descriptions.Is there anything you'd like reviewers to focus on?
Nothing in particular!