I haven't checked version 3.0.0, but version 1.3.9 used to have sources
with GPL headers and was rejected by Eclipse IP team. Probably worth to
double-check this has been cleaned up.
--
Regards,
Igor
On 2014-09-22, 12:14, Kristian Rosenvold wrote:
<dependency>
<groupId>com.google.code.findbugs</groupId>
<artifactId>jsr305</artifactId>
<version>3.0.0</version>
<scope>provided</scope>
</dependency>
The reason for this is that the findbugs project has been evolving
these annotations at a pace, and I would now also like to use
@CheckForNull, @CheckReturnValue, @OverridingMethodsMustInvokeSuper,
@WillClose and @WillNotClose.
(CheckReturnValue makes it an error NOT to check the return value,
which is good for immutable classes...)
I think these annotations are particularly valuable in shared code
(maven-shared and all of plexus).
Kristian
2014-09-22 18:10 GMT+02:00 Kristian Rosenvold <[email protected]>:
Some time ago, we discussed using the JSR305 annotations. At the time we
discussed @Nonnull and @Nullable, and it turned out that those two
annotations are "named based" in most analysis-tools; you can make your own
org.apache.maven.annotations.Nonnull/Nullable and have a reasonable chance
of having tools pick them up. Both findbugs and IntelliJ support this
approach. We sort-of concluded that this would be the optimal approach, and
since then we promptly did nothing about it :)
I'm very happy that we didn't do anything about it, since I'd now like to
re-propose that we actually use
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
