Add `access-modifier-suppressions` by default - Githubissues

jenkinsci / plugin-pom

Parent POM for Jenkins Plugins

https://wiki.jenkins-ci.org/display/JENKINS/Plugin+tutorial

MIT License

67 stars 75 forks source link

Add `access-modifier-suppressions` by default #904

Open basil opened 4 months ago

basil commented 4 months ago

What feature do you want to see added?

In both https://github.com/jenkinsci/azure-ad-plugin/pull/550 and https://github.com/jenkinsci/htmlpublisher-plugin/pull/274 plugins wanted to use access-modifier-suppressions but didn't find it working out of the box and added it in a way that bloated the JAR. Since this is a common mistake, it might be helpful for the default plugin build to include this functionality out of the box.

Upstream changes

No response

Are you interested in contributing this feature?

No response

basil commented 4 months ago

Filed in plugin-pom but this could just as easily be implemented in core by shipping the access-modifier-suppressions JAR as part of core.