Why only the top 5?

[peritus]

Here's what I do in my package.json:

{
  "private": true,
  "scripts": {
    "lint-report": "cat node_modules/eslint-detailed-reporter/lib/detailed.js|sed s/take(5)/take(10000)/ > node_modules/eslint-detailed-reporter/lib/detailed.js.fixed;mv node_modules/eslint-detailed-reporter/lib/detailed.js{.fixed,};./node_modules/eslint/bin/eslint.js src/ test/ -f node_modules/eslint-detailed-reporter/lib/detailed.js -o eslintreport.html"

to see more than the top 5 rule violations.

Reason for that is that I want to switch as many as possible from warning to error and starting at the bottom just works best for that.

Can you make the number of top violations to show configurable (yes, maybe 100 is a better default then 100000) ?

Also, would it be possible to also report the rules that do have 0 violations ?

[mportuga]

I originally chose 5, because in my own usage, I usually chose to tackle the worst case first. So I though any more than 5 would be too much.
I will see if I can make it configurable though, and just keep 5 as the default.
And what is the use case for wanting to see the rules that have no violations?

[peritus]

Thanks for making it configurable!

I have an eslintrc with some rules configured as err, some as warn. The moment the codebase has 0 violations for a rule configured as warn I want to switch it to err. That is easy if I fix the code in the same commit, but seing on the report which rules I could switch would.be great.

[davityavryan]

Hi guys,

Is this done already? can I configure to show all errors/warnings?

p.s. IMO it would be better to show all errors by default. could be configured to show top N though, but show all of them by default.

[mportuga]

You can still view all errors in the body of the files on the list bellow.

[CodySchneider]

ESLint doesn't seem to support passing arguments to formatters via command line other than through ENV vars: https://eslint.org/docs/latest/developer-guide/working-with-custom-formatters#using-environment-variables

Submitted PR to allow configuring the number of summary results.
#49