Skip to content
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

Add access-modifier-suppressions by default #904

Open
basil opened this issue Mar 13, 2024 · 1 comment
Open

Add access-modifier-suppressions by default #904

basil opened this issue Mar 13, 2024 · 1 comment

Comments

@basil
Copy link
Member

basil commented Mar 13, 2024

What feature do you want to see added?

In both jenkinsci/azure-ad-plugin#550 and jenkinsci/htmlpublisher-plugin#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
Copy link
Member Author

basil commented Mar 13, 2024

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant