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

Mark com.google.code.findbugs:annotations provided #135

Merged
merged 1 commit into from
Mar 7, 2016
Merged

Mark com.google.code.findbugs:annotations provided #135

merged 1 commit into from
Mar 7, 2016

Conversation

Stephan202
Copy link
Contributor

Rationale outlined in the commit message:

This way the "annotations" dependency does not end up on the classpath of dependant projects, which is nice for projects that e.g. use the jsr305 module instead. Note that complete absence of the annotations from the classpath does not matter at runtime.

Related is ronmamo/reflections#69, though in this case jsr305 is not a viable alternative due to the use of the SuppressFBWarnings annotation.

This way the "annotations" dependency does not end up on the classpath of
dependant projects, which is nice for projects that e.g. use the jsr305 module
instead. Note that complete absence of the annotations from the classpath does
not matter at runtime.
jqno added a commit that referenced this pull request Mar 7, 2016
Mark com.google.code.findbugs:annotations provided
@jqno jqno merged commit e69f9c6 into jqno:master Mar 7, 2016
@jqno
Copy link
Owner

jqno commented Mar 7, 2016

Good point, thanks for the PR!

@Stephan202 Stephan202 deleted the provided-annotations branch March 7, 2016 08:47
@jqno
Copy link
Owner

jqno commented Mar 13, 2016

I've just made a new release that contains this PR.

@Stephan202
Copy link
Contributor Author

Thanks, tested it and works :).

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

Successfully merging this pull request may close these issues.

2 participants