[MCHECKSTYLE-420] includes is broken
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 42/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- java
- Domain
- build-system
Research direction
Start by reproducing the reported configuration from the issue with the listed includes patterns against the affected 3.2.0 behavior. Trace how includes are interpreted and verify completion when Java files under src are checked while target files are excluded.
Written by the indexing model from the issue text.
Description
Delany opened MCHECKSTYLE-420 and commented
I thought I could achieve MCHECKSTYLE-412 using includes
<configuration>
<configLocation>${build.root}/checkstyle.xml</configLocation>
<includes>src\/**\/*.java</includes>
</configuration>
but I dont get any violations. Neither with any of
<includes>src/**/*.java</includes>
<includes>**/src/**/*.java</includes>
<includes>**\/src\/**\/*.java</includes>
I get 69 violations (files in src and target) using
<configuration>
<configLocation>${build.root}/checkstyle.xml</configLocation>
<includes>**\/*.java</includes>
</configuration>
Affects: 3.2.0
- Dominant language
- Java
- Stars
- 98
- Forks
- 69
- Avg merge
- 12h 42m
- Merged PRs (30d)
- 6
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
-
maintenance
Difficulty 2/5 1-3 hours Newbie friendliness 55/100
-
enhancement
Difficulty 5/5 Over a week Newbie friendliness 38/100
-
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
-
DefaultCheckstyleExecutor should be a failure, not an errorPossibly taken @efegokdemir claimed this 34 days ago. Openbug
Difficulty 3/5 1-2 days Newbie friendliness 58/100
apache/maven-checkstyle-plugin#656 · 1 comment ·
-
Add ability to print warnings when `failOnViolation: true` and `violationSeverity: error`Possibly taken @efegokdemir claimed this 32 days ago. Openenhancement
Difficulty 3/5 1-2 days Newbie friendliness 50/100
apache/maven-checkstyle-plugin#648 · 5 comments ·
All issues in apache/maven-checkstyle-plugin
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
strimzi/strimzi-kafka-operator#13328 · 1 comment · 3 reactions ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 67/100
OpenLiberty/ci.maven#2137 ·
Maintainers usually reply within 1 day
-
Payer adjustment can exceed the balance left after copay and deductible, so claim payments add up to more than the chargePossibly taken @DG47 claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 79/100
synthetichealth/synthea#1772 ·
Maintainers usually reply within 13 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
geonetwork/core-geonetwork#9592 ·
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
NationalSecurityAgency/ghidra#10040 ·
Maintainers usually reply within 2 days