[MCHECKSTYLE-405] The checkstyle plugin should report the number of all violations
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 35/100
- Issue type
- Feature
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- java
- Domain
- build-system
Research direction
Reproduce the two output cases using config/maven_checks.xml and the Checkstyle 8.41.1 examples in the issue. Trace the plugin's violation reporting and file-result caching entry points; done means info and warning output includes the total violations, and error output is logged at WARN level.
Written by the indexing model from the issue text.
Description
Guillaume Nodet opened MCHECKSTYLE-405 and commented
When there is an error, the build looks like:
[INFO] There is 1 error reported by Checkstyle 8.41.1 with config/maven_checks.xml ruleset.
[ERROR] src/main/java/org/apache/maven/execution/BuildResumptionDataRepository.java:[54,30] (whitespace) ParenPad: '(' is not followed by whitespace.
and when this error is fixed, the build then looks like:
[INFO] You have 0 Checkstyle violations.
The problem is that checkstyle never caches the results for files with violations, even at info or warning level. This cause the files to be always processed, without the user really being aware of any problem.
The number of violations at info or warning level should also be printed. In addition, if there are errors, the output should be logged at WARN level.
1 votes, 2 watchers
- Dominant language
- Java
- Stars
- 98
- Forks
- 72
- Avg merge
- 8h 29m
- Merged PRs (30d)
- 12
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- No contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from apache/maven-checkstyle-plugin
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
apache/maven-checkstyle-plugin#735 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 3/5 Half a day Newbie friendliness 25/100
apache/maven-checkstyle-plugin#736 · 1 comment ·
Maintainers usually reply within 1 day
-
DefaultCheckstyleExecutor uses deprecated methodPossibly taken @efegokdemir claimed this 4 days ago. Openmaintenance
Difficulty 2/5 1-3 hours Newbie friendliness 55/100
apache/maven-checkstyle-plugin#718 ·
Maintainers usually reply within 1 day
-
enhancement
Difficulty 5/5 Over a week Newbie friendliness 38/100
apache/maven-checkstyle-plugin#675 ·
Maintainers usually reply within 1 day
-
Update dependencies.May be free again A pull request for this issue was closed without being merged. Openbug
Difficulty 2/5 1-3 hours Newbie friendliness 55/100
apache/maven-checkstyle-plugin#664 ·
Maintainers usually reply within 1 day
All issues in apache/maven-checkstyle-plugin
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 66/100
a2aproject/a2a-java#1275 ·
Maintainers usually reply within 1 day
-
module/ftp Type/Bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
ballerina-platform/ballerina-library#9476 ·
Maintainers usually reply within 1 day
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
anwilli5/coin-collection-android-US#515 ·
Maintainers usually reply within 1 day
-
status: waiting-for-triage type: bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
spring-projects/spring-integration#11654 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 Under an hour Newbie friendliness 70/100
Vakore/ZappierGames#248 ·