jamesfredley opened a new issue, #16016:
URL: https://github.com/apache/grails-core/issues/16016

   ## Summary
   
   The independent `grails-gradle` build currently has 144 PMD findings, all in 
`grails-gradle-model`. The other analyzed Gradle-plugin subprojects currently 
have no PMD source task rather than a verified clean Java baseline.
   
   ## Reproduction
   
   From `grails-gradle/`:
   
   ```shell
   ./gradlew aggregateAnalysisViolations --continue \
     -Pgrails.code-analysis.enabled.pmd=true \
     -Pgrails.code-analysis.ignoreFailures=true
   ```
   
   The detailed report is 
`build/reports/code-analysis/pmd/grails-gradle-model-pmdMain.xml`.
   
   ## Baseline
   
   | Project | Findings |
   |---------|---------:|
   | `grails-gradle-model` | 144 |
   
   ## Suggested approach
   
   Resolve the model findings in reviewable rule/category batches while 
preserving public Gradle model behavior. Once the report is clean, remove 
`ignoreFailures` from the `grails-gradle` code-analysis workflow and make PMD 
blocking for `:grails-gradle-model`.
   
   ## Acceptance criteria
   
   - `grails-gradle-model-pmdMain.xml` contains zero findings.
   - The complete `grails-gradle` test suite passes.
   - PMD runs without `grails.code-analysis.ignoreFailures=true` in 
`grails-gradle` CI.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to