-
Notifications
You must be signed in to change notification settings - Fork 12
Tweak fixable documentation link #118
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
Conversation
README.md
Outdated
@@ -61,7 +61,7 @@ Generated content in a rule doc (everything above the marker comment): | |||
|
|||
💼 This rule is enabled in the following configs: `all`, ✅ `recommended`. | |||
|
|||
🔧 This rule is automatically fixable by the `--fix` [CLI option](https://eslint.org/docs/latest/user-guide/command-line-interface#--fix). | |||
🔧 This rule is automatically fixable by [the `--fix` CLI option](https://eslint.org/docs/user-guide/command-line-interface#--fix). |
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.
Can we exclude the
from the link? I don't think that needs to be part of it.
I actually got some similar feedback in this comment: sindresorhus/eslint-plugin-unicorn#1930 (comment)
You'll also need to update the text here: eslint-doc-generator/lib/rule-notices.ts Line 97 in 1e56d18
|
And run |
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.
Thanks!
We're taking about the
--fix
flag, so maybe we should just link to it's docs right away?