-
Notifications
You must be signed in to change notification settings - Fork 34
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
Support for exclusions file #198
Comments
Could you give any other plugin that has exclusions in files? This is not something easy to implement unless Maven has direct support for this. The current exclusions are a standard Maven feature for plugins. In addition like in Gradle, if a setting in a pom.xml changes the execution has to be repeated. If you see a problem with this, please open a bug report in Maven. I don't understand what you intend to do. |
Checkstyle [1][2] for example allows this. They allow a XML configuration for example <module>
...
<module name="SuppressionFilter">
<property name="file" value="suppressions.xml"/>
</module>
</module> with <suppressions>
<suppress files="SuppressedJavaFile.java" checks="SuppressedCheckId" />
</suppressions> A re-execution of your build tool should not matter in terms of dependency resolution to your container build chain if you modify your inclusions/exclusions because the dependencies will (or should) not change from that. I was not aware that the exclusion configuration is a Maven native setting. Thanks for sharing. So is there an alternative like this? mvn de.thetaphi:forbiddenapis forbiddenapis:check -Dexclude=my/package/MyClass.class -Dexclude=my/package/OtherClass.class This would work for me, too. -- |
Hi, I only do not know if or how it works with multiple patterns! Pro Tip: If you want to have your excludes in your project directory, use |
It is currently supported to set signature files that allow externalizing the linting configuration. But this is not possible with exclusions. Using containers with
maven
is considered best to cache the dependency container layer with thepom.xml
.This allows running a prebaked container image "without" requiring an internet connection for dependencies.
docker run -it $(docker build -q .) package
The need to modify the
pom.xml
just to add an exclusion would break the dependency caching.I could imagine a configuration option that would match the currently used naming like this
The text was updated successfully, but these errors were encountered: