You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jdo-dev@db.apache.org by "clr-apache (via GitHub)" <gi...@apache.org> on 2026/05/12 13:40:47 UTC
[PR] Update RAT excludes file path to work across all submodules (db-jdo)
clr-apache opened a new pull request, #124:
URL: https://github.com/apache/db-jdo/pull/124
Use \${maven.multiModuleProjectDirectory} to reference .rat-excludes from the project root so all submodules pick up the excludes configuration, not just the root module.
## Summary
The `.rat-excludes` file at the project root was only being picked up by
the root module. Submodules were ignoring it because the relative path
`.rat-excludes` resolved to each submodule's own basedir, where no such
file exists.
Fix by using `${maven.multiModuleProjectDirectory}` to reference the
excludes file from the project root in both `pom.xml` and
`parent-pom/pom.xml`, so all submodules consistently use the same
excludes list.
This ensures the `META-INF/services/*` and other exclusions added in
#123 are honoured when running `apache-rat:check` across all modules.
--
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: jdo-dev-unsubscribe@db.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Update RAT excludes file path to work across all submodules (db-jdo)
Posted by "sonarqubecloud[bot] (via GitHub)" <gi...@apache.org>.
sonarqubecloud[bot] commented on PR #124:
URL: https://github.com/apache/db-jdo/pull/124#issuecomment-4431074179
## [](https://sonarcloud.io/dashboard?id=db-jdo&pullRequest=124) **Quality Gate passed**
Issues
 [0 New issues](https://sonarcloud.io/project/issues?id=db-jdo&pullRequest=124&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)
 [0 Accepted issues](https://sonarcloud.io/project/issues?id=db-jdo&pullRequest=124&issueStatuses=ACCEPTED)
Measures
 [0 Security Hotspots](https://sonarcloud.io/project/security_hotspots?id=db-jdo&pullRequest=124&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)
 [0.0% Coverage on New Code](https://sonarcloud.io/component_measures?id=db-jdo&pullRequest=124&metric=new_coverage&view=list)
 [0.0% Duplication on New Code](https://sonarcloud.io/component_measures?id=db-jdo&pullRequest=124&metric=new_duplicated_lines_density&view=list)
<!-- sqra-placement-anchor -->
[See analysis details on SonarQube Cloud](https://sonarcloud.io/dashboard?id=db-jdo&pullRequest=124)
--
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: jdo-dev-unsubscribe@db.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Update RAT excludes file path to work across all submodules (db-jdo)
Posted by "tzaeschke (via GitHub)" <gi...@apache.org>.
tzaeschke merged PR #124:
URL: https://github.com/apache/db-jdo/pull/124
--
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: jdo-dev-unsubscribe@db.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org