apache / apache/maven-checkstyle-plugin

[MCHECKSTYLE-456] Checkstyle seems to not print info about skip

Open
#367 1 comment 0 reactions 0 assignees View on GitHub
maintenance priority:minor
Dominant language
Java
Stars
97
Forks
67
Avg merge
14h 50m
Merged PRs (30d)
6

Description

**[Matthias Bünger](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=JIRAUSER302100)** opened **[MCHECKSTYLE-456](https://issues.apache.org/jira/browse/MCHECKSTYLE-456?redirect=false)** and commented

When checkstyle plugin is skipped via skip-property it looks like the plugin does not tell the user, while other plugins do

> [INFO] --- jacoco:0.8.12:prepare-agent-integration (default) @ META-POM ---
> [INFO] Skipping JaCoCo execution because property jacoco.skip is set.
> [INFO]
> [INFO] --- failsafe:3.5.2:integration-test (default) @ META-POM ---
> [INFO] Tests are skipped.
> [INFO]
> [INFO] --- failsafe:3.5.2:verify (default) @ META-POM ---
> [INFO] Tests are skipped.
> [INFO]
> [INFO] --- checkstyle:3.6.0:check (default) @ META-POM ---
> [INFO]
> [INFO] >>> pmd:3.26.0:check (pmd_check) > :pmd @ META-POM >>>
> [INFO]
> [INFO] --- pmd:3.26.0:pmd (pmd) @ META-POM ---
> [INFO] Skipping org.apache.maven.plugins:maven-pmd-plugin:3.26.0:pmd report goal
> [INFO]
> [INFO] <<< pmd:3.26.0:check (pmd_check) < :pmd @ META-POM <<<
> [INFO]
> [INFO]
> [INFO] --- pmd:3.26.0:check (pmd_check) @ META-POM ---
> [INFO] Skipping PMD execution
> [INFO]
> [INFO] >>> pmd:3.26.0:cpd-check (cpd_check) > :cpd @ META-POM >>>
> [INFO]
> [INFO] --- pmd:3.26.0:cpd (cpd) @ META-POM ---
> [INFO] Skipping org.apache.maven.plugins:maven-pmd-plugin:3.26.0:cpd report goal
> [INFO]
> [INFO] <<< pmd:3.26.0:cpd-check (cpd_check) < :cpd @ META-POM <<<
> [INFO]
> [INFO]
> [INFO] --- pmd:3.26.0:cpd-check (cpd_check) @ META-POM ---
> [INFO] Skipping CPD execution
> [INFO]
> [INFO] >>> spotbugs:4.9.2.0:check (default) > :spotbugs @ META-POM >>>
> [INFO]
> [INFO] --- spotbugs:4.9.2.0:spotbugs (spotbugs) @ META-POM ---
> [INFO] Skipping com.github.spotbugs:spotbugs-maven-plugin:4.9.2.0:spotbugs report goal
> [INFO]
> [INFO] <<< spotbugs:4.9.2.0:check (default) < :spotbugs @ META-POM <<<
> [INFO]
> [INFO]
> [INFO] --- spotbugs:4.9.2.0:check (default) @ META-POM ---
> [INFO] Spotbugs plugin skipped
> [INFO]
> [INFO] --- jacoco:0.8.12:report (default) @ META-POM ---
> [INFO] Skipping JaCoCo execution because property jacoco.skip is set.

---

**Affects:** 3.6.0

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the Maven Checkstyle plugin's handling of the skip-property and the path that produces its execution output. Compare the skipped Checkstyle output with the examples in the issue, then add coverage for the expected informational message if the repository has a relevant test area. Done means a skipped execution clearly reports that it was skipped.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
build-system
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.