Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign upPromote --fix when standard detects issues #576
Comments
This comment has been minimized.
This comment has been minimized.
|
Great idea -- this is easy to add in |
feross
closed this
Jul 26, 2016
lock
bot
locked as resolved and limited conversation to collaborators
May 10, 2018
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
timoxley commentedJul 26, 2016
•
edited
Problem
standard --fixdoes a lot of hard work for the user and should reduce instances of linter-rage when faced with sweeping changes, such as the new JSX single -> double quote rule: standard/eslint-config-standard#27.However,
standard --fixis new and many users will not know to even look for it. Some recent anecdotal evidence:This new & handy feature could do with more promotion, especially when it's most applicable: when the user is staring at a wall of linter errors.
Suggestion
Mention
--fixwhenever standard reports an error.Issue stemming from this comment: #564 (comment)
I'd submit a PR but it's not obvious what the wording should be or how this should be achieved.
Ideally the error message would span multiple lines like:
But that's probably not possible in current setup as is. The tagline is printed in between
standard:and the(homepage)in the following:Without looking too deeply it looks like the formatting of the error message is applied by eslint. Perhaps there are other options to get it to print a different thing?
Embedding it on the one line as a part of the tagline looks like:
This is maybe just acceptable in this form, though it is a bit wordy.
The tagline is not really the right place for this as it also infects
--help(and perhaps other places?):Maybe this is ok though?
Overall:
¯\_(ツ)_/¯