Repository navigation
build: Bump dependencies - #67
Conversation
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Coverage variation | ✅ +0.00% coverage variation (-1.00%) |
| Diff coverage | ✅ ∅ diff coverage |
Coverage variation details
Coverable lines Covered lines Coverage Common ancestor commit (74adfb4) 195 195 100.00% Head commit (e4eec4b) 195 (+0) 195 (+0) 100.00% (+0.00%) Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch:
<coverage of head commit> - <coverage of common ancestor commit>
Diff coverage details
Coverable lines Covered lines Diff coverage Pull request (#67) 0 0 ∅ (not applicable) Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified:
<covered lines added or modified>/<coverable lines added or modified> * 100%
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
Uses the shared PMD ruleset from nva-gradle-plugins with repository exclusions, instead of a full local override. Test sources stay out of PMD as before. GuardLogStatement is excluded as in the other repositories, and UnnecessaryLocalBeforeReturn becomes VariableCanBeInlined, its replacement in PMD 7.
c291dcf to
e4eec4b
Compare
| } | ||
| pmd { | ||
| rulesetFile = rootProject.file('config/pmd/ruleset.xml') | ||
| analyzeTests = false |
There was a problem hiding this comment.
This repo was configured with <exclude-pattern>.*/test/.*</exclude-pattern>, which means the test code is likely littered with PMD violations. This setting effectively does the same thing.
We should fix this, but not now...
Routine dependency bump.
Changes:
Worth a look:
This turns on the
bestpractices,multithreadingandperformancecategories for main code, while test sources stay out of PMD as before.GuardLogStatementis excluded as in the other repositories, andUnnecessaryLocalBeforeReturn(deprecated in PMD 7) is replaced by its successorVariableCanBeInlined.