-
Notifications
You must be signed in to change notification settings - Fork 202
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 build #468
Clean up build #468
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -68,45 +68,37 @@ | |
<dependency> | ||
<groupId>com.github.spotbugs</groupId> | ||
<artifactId>spotbugs-annotations</artifactId> | ||
<optional>true</optional> | ||
<exclusions> | ||
<exclusion> | ||
<groupId>com.google.code.findbugs</groupId> | ||
<artifactId>jsr305</artifactId> | ||
</exclusion> | ||
</exclusions> | ||
Comment on lines
+72
to
+77
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Consistent with other repositories, ban JSR 305. |
||
</dependency> | ||
<dependency> | ||
<groupId>com.github.tomakehurst</groupId> | ||
<artifactId>wiremock-jre8</artifactId> | ||
<version>2.33.2</version> | ||
<scope>test</scope> | ||
<exclusions> | ||
<exclusion> | ||
<!-- Rely on jackson-annotations from jackson-dataformat-yaml --> | ||
<groupId>com.fasterxml.jackson.core</groupId> | ||
<artifactId>jackson-annotations</artifactId> | ||
</exclusion> | ||
<exclusion> | ||
<!-- Rely on jackson-databind from jackson-dataformat-yaml --> | ||
<groupId>com.fasterxml.jackson.core</groupId> | ||
<artifactId>jackson-databind</artifactId> | ||
</exclusion> | ||
</exclusions> | ||
<groupId>com.github.tomakehurst</groupId> | ||
<artifactId>wiremock-jre8</artifactId> | ||
<version>2.33.2</version> | ||
<scope>test</scope> | ||
<exclusions> | ||
<exclusion> | ||
<!-- Rely on jackson-annotations from jackson-dataformat-yaml --> | ||
<groupId>com.fasterxml.jackson.core</groupId> | ||
<artifactId>jackson-annotations</artifactId> | ||
</exclusion> | ||
<exclusion> | ||
<!-- Rely on jackson-databind from jackson-dataformat-yaml --> | ||
<groupId>com.fasterxml.jackson.core</groupId> | ||
<artifactId>jackson-databind</artifactId> | ||
</exclusion> | ||
</exclusions> | ||
</dependency> | ||
<dependency> | ||
<groupId>com.github.stefanbirkner</groupId> | ||
<artifactId>system-rules</artifactId> | ||
<version>1.19.0</version> | ||
<scope>test</scope> | ||
</dependency> | ||
<groupId>com.github.stefanbirkner</groupId> | ||
<artifactId>system-rules</artifactId> | ||
<version>1.19.0</version> | ||
<scope>test</scope> | ||
</dependency> | ||
Comment on lines
+80
to
+102
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fixing incorrect indentation. |
||
</dependencies> | ||
|
||
<build> | ||
<pluginManagement> | ||
<plugins> | ||
<plugin> | ||
<groupId>org.apache.maven.plugins</groupId> | ||
<artifactId>maven-javadoc-plugin</artifactId> | ||
<configuration> | ||
<source>8</source> | ||
</configuration> | ||
</plugin> | ||
</plugins> | ||
</pluginManagement> | ||
</build> | ||
|
||
Comment on lines
-98
to
-111
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Redundant; already in the grandparent POM. |
||
</project> |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,12 +5,14 @@ | |
<parent> | ||
<groupId>org.jenkins-ci</groupId> | ||
<artifactId>jenkins</artifactId> | ||
<version>1.85</version> | ||
<version>1.86</version> | ||
<relativePath /> | ||
</parent> | ||
|
||
<properties> | ||
<revision>2.12.9</revision> | ||
<changelist>-SNAPSHOT</changelist> | ||
<gitHubRepo>jenkinsci/plugin-installation-manager-tool</gitHubRepo> | ||
<spotbugs.excludeFilterFile>${maven.multiModuleProjectDirectory}/src/spotbugs/spotbugs-excludes.xml</spotbugs.excludeFilterFile> | ||
<tagNameFormat>@{project.version}</tagNameFormat> | ||
</properties> | ||
|
@@ -25,18 +27,17 @@ | |
<module>plugin-management-library</module> | ||
<module>plugin-management-cli</module> | ||
</modules> | ||
|
||
<scm> | ||
<connection>scm:git:ssh://git@github.com/jenkinsci/plugin-installation-manager-tool.git</connection> | ||
<developerConnection>scm:git:ssh://git@github.com/jenkinsci/plugin-installation-manager-tool.git</developerConnection> | ||
<url>https://github.com/jenkinsci/plugin-installation-manager-tool</url> | ||
<scm child.scm.connection.inherit.append.path="false" child.scm.developerConnection.inherit.append.path="false" child.scm.url.inherit.append.path="false"> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Good for multi-module Maven projects. |
||
<connection>scm:git:https://github.com/${gitHubRepo}.git</connection> | ||
<developerConnection>scm:git:git@github.com:${gitHubRepo}.git</developerConnection> | ||
<url>https://github.com/${gitHubRepo}</url> | ||
Comment on lines
+31
to
+33
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Incrementalified. |
||
<tag>${scmTag}</tag> | ||
</scm> | ||
|
||
<licenses> | ||
<license> | ||
<name>The MIT license</name> | ||
<url>http://www.opensource.org/licenses/mit-license.php</url> | ||
<url>https://opensource.org/licenses/MIT</url> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Current URL with HTTPS. |
||
<distribution>repo</distribution> | ||
</license> | ||
</licenses> | ||
|
@@ -57,17 +58,6 @@ | |
|
||
<dependencyManagement> | ||
<dependencies> | ||
<dependency> | ||
<groupId>junit</groupId> | ||
<artifactId>junit</artifactId> | ||
<version>4.13.2</version> | ||
<exclusions> | ||
<exclusion> | ||
<groupId>org.hamcrest</groupId> | ||
<artifactId>hamcrest-core</artifactId> | ||
</exclusion> | ||
</exclusions> | ||
</dependency> | ||
Comment on lines
-60
to
-70
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Redundant; in the parent POM. |
||
<dependency> | ||
<groupId>org.assertj</groupId> | ||
<artifactId>assertj-core</artifactId> | ||
|
@@ -78,13 +68,6 @@ | |
<artifactId>hamcrest-core</artifactId> | ||
<version>2.2</version> | ||
</dependency> | ||
<dependency> | ||
<groupId>org.junit</groupId> | ||
<artifactId>junit-bom</artifactId> | ||
<version>5.9.0</version> | ||
<type>pom</type> | ||
<scope>import</scope> | ||
</dependency> | ||
Comment on lines
-81
to
-87
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Redundant; in the parent POM. |
||
<dependency> | ||
<groupId>org.slf4j</groupId> | ||
<artifactId>slf4j-api</artifactId> | ||
|
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.
EOL JSR 305 as we have in other repositories; switch to SpotBugs JAR.