-
-
Notifications
You must be signed in to change notification settings - Fork 24
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
Sort by severity first? #8
Comments
👍 Should still be ordered by line number per warning and error group though.
👍 |
Merged
sindresorhus
pushed a commit
that referenced
this issue
Sep 16, 2016
* Sort results by severity Fixes #8. * Lines with only warnings should be listed first
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Sorting by line number makes a lot of sense, but I ignore most warnings anyway (or rather, most warnings I see are ones I evaluated and decided to ignore at some point in the past).
I think it might make sense to put the higher severity
error
at the bottom of the list (since we are scrolling up in our terminal to view it anyways.Also, it might make sense to reorder the files so those with only warnings and no errors come first.
This gets that one error you missed down at the bottom of the output every time.
The text was updated successfully, but these errors were encountered: