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

Clean up some eclipse warnings #386

merged 5 commits into from Nov 12, 2018


None yet
2 participants
Copy link

commented Nov 2, 2018

No description provided.

dpursehouse added some commits Nov 2, 2018

Use Integer.valueOf instead of deprecated Integer(int) constructor
Change-Id: Ic510be94fcd68159a1ee2104e01d9aec30bfc781
Signed-off-by: David Pursehouse <>
Add missing serialVersionUID on classes that implement Serializable
Change-Id: I6fee2d80d03b592bd91f333c68e3ad6d5738938b
Signed-off-by: David Pursehouse <>
Change-Id: Ifdab7bb446cab97a5986de858a38219f4ce5f60b
Signed-off-by: David Pursehouse <>
Revert "Upgrade animal-maven-sniffer-plugin to 1.17"
This plugin's version is managed by the parent pom.xml, and changing
it causes an "Overriding managed version" warning.

This reverts commit cc2d980.

Change-Id: Iced06e660e2eeeebbe87db1c4861846dd34e09e5
Signed-off-by: David Pursehouse <>
Don't specify version for animal-sniffer-maven-plugin
The version of this plugin is specified in the parent pom.xml,
and setting it here (to the same version) causes a "Duplicating
managed version" warning.

Change-Id: I347b1840c254a9b0f3385d0d1bd9ffbd95cda2c1
Signed-off-by: David Pursehouse <>

@rsandell rsandell merged commit 5748ac9 into jenkinsci:master Nov 12, 2018

1 check passed

continuous-integration/jenkins/pr-merge This commit looks good
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.