Print line violations only under --verbose flag #49
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Alleviates #48 but a proper performance fix would be better. Also,
--verbose
already existed, so I reused it. Maybe, it's a bad idea, 'cause it will be impossible to find anything in a string of[ Checked ]
(this is what--verbose
currently means: it prints this for all non-violating files). But that's what was suggested in #48… Opinions?Otherwise, I put back the old implementation beside the new one, mostly. Some common bits are factors out.
Time measurements