-
-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix #79 - Use plexus to support classpath resources #80
Fix #79 - Use plexus to support classpath resources #80
Conversation
This is inspired by the PMD maven plugin https://github.com/apache/maven-pmd-plugin/blob/4afad98d229a6764d2ebdd246c037d1645c7cbb9/src/main/java/org/apache/maven/plugins/pmd/PmdReport.java#L394 The aim is to support classpath configuration when using this plugin in parent aggregator projects where config is in a plugin dependency
4a61d87
to
53c60b0
Compare
@jonathanlermitage The tests are passing locally for me but I'm not sure why they are failing here. Is there a way to pull the Maven |
Figured it out 😄 |
2ad7f56
to
ce35df4
Compare
Thanks! |
Hum, Maven makes some resistance, again 😅 I will try to publish the new 1.9.0 release asap.
(my ossrh maven profile is configured with my Sonatype creds, and my gpg passphrase)
It will simply take some time... Nota: I received a Notice of Changes to OSSRH (Sonatype) Account Management. I will look at this. Edit: okayyyy, I can't login to Nexus, and changing my password doesn't work... I will contact their support team. |
Ooh that sounds rough @jonathanlermitage. I'm going to guess that because it's login related there is not much I can do to help, but if there is let me know! Good luck! |
Thx! The Sonatype support helped me a lot. I just published the 1.9.2 release (strictly identical to 1.9.0) with success. It should be available in Central very soon. 🥳 |
Fixes #79
The code here is inspired by the PMD maven plugin
https://github.com/apache/maven-pmd-plugin/blob/4afad98d229a6764d2ebdd246c037d1645c7cbb9/src/main/java/org/apache/maven/plugins/pmd/PmdReport.java#L394
The aim is to support classpath configuration when using this plugin in parent aggregator projects where config is in a plugin dependency.