Skip to content

Spotbugs: NP_NONNULL_PARAM_VIOLATION#8129

Merged
leventov merged 2 commits intoapache:masterfrom
Fokko:NP_NONNULL_PARAM_VIOLATION
Aug 2, 2019
Merged

Spotbugs: NP_NONNULL_PARAM_VIOLATION#8129
leventov merged 2 commits intoapache:masterfrom
Fokko:NP_NONNULL_PARAM_VIOLATION

Conversation

@Fokko
Copy link
Copy Markdown
Contributor

@Fokko Fokko commented Jul 22, 2019

Enables NP_NONNULL_PARAM_VIOLATION in Spotbugs

@Fokko
Copy link
Copy Markdown
Contributor Author

Fokko commented Jul 23, 2019

@Fokko
Copy link
Copy Markdown
Contributor Author

Fokko commented Jul 26, 2019

Fixed conflicts

@Fokko Fokko force-pushed the NP_NONNULL_PARAM_VIOLATION branch from b707bcd to 2efc47a Compare July 30, 2019 20:07
@Fokko
Copy link
Copy Markdown
Contributor Author

Fokko commented Jul 30, 2019

Rebased as well

@Fokko Fokko force-pushed the NP_NONNULL_PARAM_VIOLATION branch from 690abc6 to 23df81d Compare July 31, 2019 15:22
@Fokko Fokko force-pushed the NP_NONNULL_PARAM_VIOLATION branch from 23df81d to 4a88f44 Compare July 31, 2019 15:24
@Fokko
Copy link
Copy Markdown
Contributor Author

Fokko commented Jul 31, 2019

@leventov I've removed all the unrelated changes and rebased against latest master. Let me know what you think. Would be great to get this in so I can continue on fixing all the [Not]Null annotations.

@Fokko
Copy link
Copy Markdown
Contributor Author

Fokko commented Aug 1, 2019

I guess a flakey test, it passed on my branch: https://travis-ci.org/Fokko/druid/builds/566424524

@asdf2014
Copy link
Copy Markdown
Member

asdf2014 commented Aug 1, 2019

@Fokko I just restarted the failed travis job.

@leventov leventov merged commit 91743ee into apache:master Aug 2, 2019
@Fokko Fokko deleted the NP_NONNULL_PARAM_VIOLATION branch August 2, 2019 16:28
@clintropolis clintropolis added this to the 0.16.0 milestone Aug 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants