refactor(lint): move lint rule docs to cli/tools/lint/docs/ #27293
Conversation
There was a problem hiding this comment.
This is not yet automated and tested, waiting on #27162 to land so appropriate APIs are available.
There was a problem hiding this comment.
I don't think we should store these docs here. They're docs are for lint rules that live in deno_lint, so they should either live there or we should move all these docs to the deno-docs repo. Edit: It probably makes most sense to move these to deno-docs because these are for deno-docs.
| @@ -302,6 +302,7 @@ pub struct LintFlags { | |||
| pub json: bool, | |||
| pub compact: bool, | |||
| pub watch: Option<WatchFlags>, | |||
| pub internal_print_all_rules: bool, | |||
There was a problem hiding this comment.
I don't think we need this? We can add a unit test that does this linting in tools/lint?
There was a problem hiding this comment.
Not until #27245 lands, there's no way to get all available rules right now and print them.
There was a problem hiding this comment.
I think we can use LintRuleProvider and instead write a unit test?
deno/cli/tools/lint/rules/mod.rs
Line 146 in d99b2d6
|
Closing in favor of #27324 and another PR that will be opened in |
Also adds an internal (hidden) flag to get a list of all rules,
and adds tests to ensure that all lint rules have docs as
well as the schema files are up to date.