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
Refresh plugin #291
Refresh plugin #291
Conversation
@@ -25,7 +26,8 @@ under the License. | |||
<parent> | |||
<groupId>org.jenkins-ci.plugins</groupId> | |||
<artifactId>plugin</artifactId> | |||
<version>4.49</version> | |||
<version>4.50</version> | |||
<relativePath /> |
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.
Consistent with the archetype.
@@ -35,7 +37,7 @@ under the License. | |||
<url>https://github.com/jenkinsci/${project.artifactId}-plugin</url> | |||
|
|||
<scm> | |||
<connection>scm:git:git://github.com/${gitHubRepo}.git</connection> | |||
<connection>scm:git:https://github.com/${gitHubRepo}.git</connection> |
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.
GitHub has deprecated git://
URLs.
@@ -219,30 +221,18 @@ under the License. | |||
<dependency> | |||
<groupId>org.jenkins-ci.plugins</groupId> | |||
<artifactId>jackson2-api</artifactId> | |||
<version>2.13.4.20221013-295.v8e29ea_354141</version> |
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.
Managed in plugin BOM.
<dependency> | ||
<groupId>org.hamcrest</groupId> | ||
<artifactId>java-hamcrest</artifactId> | ||
<version>2.0.0.0</version> | ||
<scope>test</scope> | ||
</dependency> |
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.
Managed in plugin parent POM.
<dependency> | ||
<groupId>junit</groupId> | ||
<artifactId>junit</artifactId> | ||
<scope>test</scope> | ||
</dependency> |
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.
Managed in plugin parent POM.
<dependency> | ||
<groupId>org.cryptacular</groupId> | ||
<artifactId>cryptacular</artifactId> | ||
</dependency> |
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.
No version number, so dead code.
catch (NullPointerException ne) { | ||
return FormValidation.warning(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.
This whole method appears to be dead code, but it trips up new versions of SpotBugs. Anyway it does not appear that a NullPointerException
could actually occur here even if this method was invoked.
CC @kuisathaverat