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
Add meta.docs
for each rule
#246
Changes from all commits
39fbe48
ec2b50e
34148ba
64d68a2
f7ea000
be53f4a
cf20a03
1c1df4c
b46768f
ce6868a
82725a4
ef50275
6e252c6
5b1e8be
34246d5
25edfdd
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20,10 +20,17 @@ | |
|
||
import { TSESTree } from '@typescript-eslint/experimental-utils'; | ||
import { Rule } from '../utils/types'; | ||
import docsUrl from '../utils/docs-url'; | ||
|
||
const rule: Rule.RuleModule = { | ||
meta: { | ||
type: 'suggestion', | ||
docs: { | ||
description: '"if ... else if" constructs should end with "else" clauses', | ||
category: 'Best Practices', | ||
recommended: false, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. should it be There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
No, it shouldn't unfortunately. Here is the type of the
Yes, we can. I will do that. |
||
url: docsUrl(__filename), | ||
}, | ||
}, | ||
create(context: Rule.RuleContext) { | ||
return { | ||
|
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.
since this forEach has a full body now, we probably can merge it with the previous one