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
ZOOKEEPER-3351 - Fix findBugs and rat check, add filename to sha #907
Conversation
Looks good. Please add the spotbugs ignore. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good. Add spotbug and we are done
Tested spotBugs, it now passes with versions without SNAPSHOT as well. Testing this patch on 3.5 branch to see if it can be merged or new PR is required. Edit: This PR fixes 3.5 branch as well, but it is conflicting, so I'll create a PR for 3.5 anyway. |
@@ -41,6 +42,8 @@ public static String getBuildDate() { | |||
return BUILD_DATE; | |||
} | |||
|
|||
@SuppressFBWarnings(value = "RCN_REDUNDANT_NULLCHECK_OF_NULL_VALUE", | |||
justification = "If SNAPSHOT is missing from version number this is redundant") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's say "QUALIFIER" instead of "SNAPSHOT".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rephased the message
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
retest ant build |
Committed to master branch. Thanks @nkalmar ! |
Note: this is only for master, for 3.5 branch it needs a different PR! Correction: I will test this on 3.5 branch first! Author: Norbert Kalmar <nkalmar@yahoo.com> Reviewers: andor@apache.org Closes apache#907 from nkalmar/ZK-3351 and squashes the following commits: ca77153 [Norbert Kalmar] ZOOKEEPER-3351 - fix findBugs issue on released versions 161df0b [Norbert Kalmar] ZOOKEEPER-3351 - add rat exceptions and fix sha generation
Note: this is only for master, for 3.5 branch it needs a different PR!
Correction: I will test this on 3.5 branch first!