diff --git a/pom.xml b/pom.xml
index 6297f630..584bbcbd 100644
--- a/pom.xml
+++ b/pom.xml
@@ -12,7 +12,7 @@
org.springframework.cloud
spring-cloud-build
- 2.2.3.RELEASE
+ 3.1.0
@@ -30,17 +30,17 @@
UTF-8
1.8
- Hoxton.SR6
- 1.0
+ 2021.0.0
+ 1.1.2
0.0.9
- 5.4.1.Final
- 5.1.3.201810200350-r
+ 7.0.2.Final
+ 6.0.0.202111291000-r
- 3.6.3
+ 3.8.4
2.7
- 4.2.0
- 0.10.0-SNAPSHOT
- 4.0.3
+ 4.3.0
+ 0.12.0
+ 4.1.1
1.0.0.BUILD-SNAPSHOT
1.1.4
5.0.4
diff --git a/projects/spring-cloud/src/test/java/releaser/cloud/buildsystem/SpringCloudCustomMavenBomTests.java b/projects/spring-cloud/src/test/java/releaser/cloud/buildsystem/SpringCloudCustomMavenBomTests.java
index e988b750..be2711da 100644
--- a/projects/spring-cloud/src/test/java/releaser/cloud/buildsystem/SpringCloudCustomMavenBomTests.java
+++ b/projects/spring-cloud/src/test/java/releaser/cloud/buildsystem/SpringCloudCustomMavenBomTests.java
@@ -21,7 +21,7 @@ import java.util.HashSet;
import java.util.List;
import java.util.Set;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import releaser.cloud.SpringCloudReleaserProperties;
import releaser.internal.buildsystem.CustomBomParser;
import releaser.internal.buildsystem.VersionsFromBom;
diff --git a/projects/spring-cloud/src/test/java/releaser/cloud/buildsystem/SpringCloudMavenBomParserTests.java b/projects/spring-cloud/src/test/java/releaser/cloud/buildsystem/SpringCloudMavenBomParserTests.java
index 717ae82c..17311f6b 100644
--- a/projects/spring-cloud/src/test/java/releaser/cloud/buildsystem/SpringCloudMavenBomParserTests.java
+++ b/projects/spring-cloud/src/test/java/releaser/cloud/buildsystem/SpringCloudMavenBomParserTests.java
@@ -22,9 +22,9 @@ import java.net.URISyntaxException;
import java.util.Arrays;
import org.eclipse.jgit.api.errors.GitAPIException;
-import org.junit.Before;
import org.junit.Rule;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
import org.junit.rules.TemporaryFolder;
import releaser.cloud.SpringCloudReleaserProperties;
import releaser.cloud.docs.TestUtils;
@@ -52,7 +52,7 @@ public class SpringCloudMavenBomParserTests {
ReleaserProperties properties = SpringCloudReleaserProperties.get();
- @Before
+ @BeforeEach
public void setup() throws URISyntaxException, IOException, GitAPIException {
this.tmpFolder = this.tmp.newFolder();
TestUtils.prepareLocalRepo();
diff --git a/projects/spring-cloud/src/test/java/releaser/cloud/buildsystem/SpringCloudProjectPomUpdaterTests.java b/projects/spring-cloud/src/test/java/releaser/cloud/buildsystem/SpringCloudProjectPomUpdaterTests.java
index ef0ef6d3..77d2cc2e 100644
--- a/projects/spring-cloud/src/test/java/releaser/cloud/buildsystem/SpringCloudProjectPomUpdaterTests.java
+++ b/projects/spring-cloud/src/test/java/releaser/cloud/buildsystem/SpringCloudProjectPomUpdaterTests.java
@@ -21,7 +21,7 @@ import java.util.Map;
import java.util.stream.Collectors;
import org.assertj.core.api.BDDAssertions;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import releaser.cloud.SpringCloudReleaserProperties;
import releaser.internal.ReleaserProperties;
import releaser.internal.buildsystem.MavenBomParserAccessor;
diff --git a/projects/spring-cloud/src/test/java/releaser/cloud/docs/SpringCloudCustomProjectDocumentationUpdaterTests.java b/projects/spring-cloud/src/test/java/releaser/cloud/docs/SpringCloudCustomProjectDocumentationUpdaterTests.java
index bf7dcf57..1da8166c 100644
--- a/projects/spring-cloud/src/test/java/releaser/cloud/docs/SpringCloudCustomProjectDocumentationUpdaterTests.java
+++ b/projects/spring-cloud/src/test/java/releaser/cloud/docs/SpringCloudCustomProjectDocumentationUpdaterTests.java
@@ -26,9 +26,9 @@ import java.util.Collections;
import javax.validation.constraints.NotNull;
import org.assertj.core.api.BDDAssertions;
-import org.junit.Before;
import org.junit.Rule;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
import org.junit.rules.TemporaryFolder;
import org.mockito.BDDMockito;
import releaser.cloud.SpringCloudReleaserProperties;
@@ -63,7 +63,7 @@ public class SpringCloudCustomProjectDocumentationUpdaterTests {
ReleaserProperties properties = SpringCloudReleaserProperties.get();
- @Before
+ @BeforeEach
public void setup() throws IOException, URISyntaxException {
this.tmpFolder = this.tmp.newFolder();
this.project = new File(SpringCloudCustomProjectDocumentationUpdater.class
diff --git a/projects/spring-cloud/src/test/java/releaser/cloud/github/SpringCloudGithubIssuesTests.java b/projects/spring-cloud/src/test/java/releaser/cloud/github/SpringCloudGithubIssuesTests.java
index 59f30a75..298fb5c4 100644
--- a/projects/spring-cloud/src/test/java/releaser/cloud/github/SpringCloudGithubIssuesTests.java
+++ b/projects/spring-cloud/src/test/java/releaser/cloud/github/SpringCloudGithubIssuesTests.java
@@ -25,9 +25,9 @@ import com.jcabi.github.Issue;
import com.jcabi.github.Repo;
import com.jcabi.github.Repos;
import com.jcabi.github.mock.MkGithub;
-import org.junit.Before;
import org.junit.Rule;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
import org.junit.rules.TemporaryFolder;
import org.mockito.BDDMockito;
import releaser.cloud.SpringCloudReleaserProperties;
@@ -53,7 +53,7 @@ public class SpringCloudGithubIssuesTests {
Repo repo;
- @Before
+ @BeforeEach
public void setup() throws IOException {
this.github = github("spring-guides");
this.properties.getGit().setOauthToken("a");
@@ -81,7 +81,7 @@ public class SpringCloudGithubIssuesTests {
"2.0.0.BUILD-SNAPSHOT")),
new ProjectVersion("sc-release", "Edgware.BUILD-SNAPSHOT"));
- BDDMockito.then(github).shouldHaveZeroInteractions();
+ BDDMockito.then(github).shouldHaveNoInteractions();
}
@Test
@@ -133,7 +133,7 @@ public class SpringCloudGithubIssuesTests {
new ProjectVersion("spring-cloud-build", "2.0.0.RELEASE")),
new ProjectVersion("sc-release", "Edgware.BUILD-SNAPSHOT"));
- BDDMockito.then(github).shouldHaveZeroInteractions();
+ BDDMockito.then(github).shouldHaveNoInteractions();
}
@Test
diff --git a/projects/spring-cloud/src/test/java/releaser/cloud/spring/AbstractSpringCloudAcceptanceTests.java b/projects/spring-cloud/src/test/java/releaser/cloud/spring/AbstractSpringCloudAcceptanceTests.java
index 2d5c9609..aaf39b6b 100644
--- a/projects/spring-cloud/src/test/java/releaser/cloud/spring/AbstractSpringCloudAcceptanceTests.java
+++ b/projects/spring-cloud/src/test/java/releaser/cloud/spring/AbstractSpringCloudAcceptanceTests.java
@@ -19,7 +19,7 @@ package releaser.cloud.spring;
import java.io.File;
import java.util.Arrays;
-import org.junit.Before;
+import org.junit.jupiter.api.BeforeEach;
import releaser.internal.buildsystem.TestUtils;
import releaser.internal.spring.AbstractSpringAcceptanceTests;
@@ -33,7 +33,7 @@ public class AbstractSpringCloudAcceptanceTests extends AbstractSpringAcceptance
public File springCloudBuildProject;
- @Before
+ @BeforeEach
public void setupCloud() throws Exception {
this.temporaryFolder = this.tmp.newFolder();
this.springCloudConsulProject = new File(AbstractSpringAcceptanceTests.class
diff --git a/projects/spring-cloud/src/test/java/releaser/cloud/spring/meta/AbstractSpringCloudMetaAcceptanceTests.java b/projects/spring-cloud/src/test/java/releaser/cloud/spring/meta/AbstractSpringCloudMetaAcceptanceTests.java
index cde22ce3..98d687bd 100644
--- a/projects/spring-cloud/src/test/java/releaser/cloud/spring/meta/AbstractSpringCloudMetaAcceptanceTests.java
+++ b/projects/spring-cloud/src/test/java/releaser/cloud/spring/meta/AbstractSpringCloudMetaAcceptanceTests.java
@@ -19,7 +19,7 @@ package releaser.cloud.spring.meta;
import java.io.File;
import java.util.Arrays;
-import org.junit.Before;
+import org.junit.jupiter.api.BeforeEach;
import releaser.internal.spring.meta.AbstractSpringMetaReleaseAcceptanceTests;
import static org.assertj.core.api.BDDAssertions.then;
@@ -31,7 +31,7 @@ public class AbstractSpringCloudMetaAcceptanceTests
public File springCloudBuildProject;
- @Before
+ @BeforeEach
public void setupCloud() throws Exception {
this.springCloudConsulProject = new File(
AbstractSpringCloudMetaAcceptanceTests.class
diff --git a/projects/spring-cloud/src/test/java/releaser/cloud/spring/meta/SpringMetaReleaseAcceptanceTests.java b/projects/spring-cloud/src/test/java/releaser/cloud/spring/meta/SpringMetaReleaseAcceptanceTests.java
index 9fafc02c..6e745b74 100644
--- a/projects/spring-cloud/src/test/java/releaser/cloud/spring/meta/SpringMetaReleaseAcceptanceTests.java
+++ b/projects/spring-cloud/src/test/java/releaser/cloud/spring/meta/SpringMetaReleaseAcceptanceTests.java
@@ -21,7 +21,7 @@ import java.util.Collections;
import java.util.LinkedHashMap;
import java.util.Map;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import org.mockito.BDDMockito;
import releaser.internal.Releaser;
import releaser.internal.ReleaserProperties;
diff --git a/projects/spring-cloud/src/test/java/releaser/cloud/spring/single/SpringSingleProjectAcceptanceTests.java b/projects/spring-cloud/src/test/java/releaser/cloud/spring/single/SpringSingleProjectAcceptanceTests.java
index 8a0e25a7..bbd46cd5 100644
--- a/projects/spring-cloud/src/test/java/releaser/cloud/spring/single/SpringSingleProjectAcceptanceTests.java
+++ b/projects/spring-cloud/src/test/java/releaser/cloud/spring/single/SpringSingleProjectAcceptanceTests.java
@@ -22,7 +22,7 @@ import java.util.Iterator;
import org.assertj.core.api.BDDAssertions;
import org.eclipse.jgit.revwalk.RevCommit;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import org.mockito.BDDMockito;
import releaser.cloud.spring.AbstractSpringCloudAcceptanceTests;
import releaser.internal.ReleaserProperties;
diff --git a/releaser-core/pom.xml b/releaser-core/pom.xml
index 44fd792e..9939da45 100644
--- a/releaser-core/pom.xml
+++ b/releaser-core/pom.xml
@@ -48,6 +48,11 @@
jsch.agentproxy.sshagent
${jsch-agent.version}
+
+ org.eclipse.jgit
+ org.eclipse.jgit.ssh.jsch
+ ${org.eclipse.jgit-version}
+
com.jcraft
jsch.agentproxy.jsch
diff --git a/releaser-core/src/main/java/releaser/internal/Releaser.java b/releaser-core/src/main/java/releaser/internal/Releaser.java
index 859abba4..cacb48d2 100644
--- a/releaser-core/src/main/java/releaser/internal/Releaser.java
+++ b/releaser-core/src/main/java/releaser/internal/Releaser.java
@@ -69,12 +69,9 @@ public class Releaser {
private ReleaserProperties releaserProperties;
- public Releaser(ReleaserProperties releaserProperties,
- ProjectPomUpdater projectPomUpdater,
- ProjectCommandExecutor projectCommandExecutor,
- ProjectGitHandler projectGitHandler,
- ProjectGitHubHandler projectGitHubHandler,
- TemplateGenerator templateGenerator, GradleUpdater gradleUpdater,
+ public Releaser(ReleaserProperties releaserProperties, ProjectPomUpdater projectPomUpdater,
+ ProjectCommandExecutor projectCommandExecutor, ProjectGitHandler projectGitHandler,
+ ProjectGitHubHandler projectGitHubHandler, TemplateGenerator templateGenerator, GradleUpdater gradleUpdater,
SaganUpdater saganUpdater, DocumentationUpdater documentationUpdater,
PostReleaseActions postReleaseActions) {
this.releaserProperties = releaserProperties;
@@ -101,56 +98,51 @@ public class Releaser {
return this.projectPomUpdater.fixedVersions();
}
- public ExecutionResult updateProjectFromBom(File project, Projects versions,
- ProjectVersion versionFromBom) {
+ public ExecutionResult updateProjectFromBom(File project, Projects versions, ProjectVersion versionFromBom) {
return updateProjectFromBom(project, versions, versionFromBom, ASSERT_SNAPSHOTS);
}
- private ExecutionResult updateProjectFromBom(File project, Projects versions,
- ProjectVersion versionFromBom, boolean assertSnapshots) {
+ private ExecutionResult updateProjectFromBom(File project, Projects versions, ProjectVersion versionFromBom,
+ boolean assertSnapshots) {
log.info("Will update the project with versions [{}]", versions);
- ReleaserProperties updatedProperties = new ReleaserPropertiesUpdater()
- .updateProperties(this.releaserProperties, project);
- this.projectPomUpdater.updateProjectFromReleaseTrain(project, versions,
- versionFromBom, assertSnapshots);
- this.gradleUpdater.updateProjectFromReleaseTrain(updatedProperties, project,
- versions, versionFromBom, assertSnapshots);
+ ReleaserProperties updatedProperties = new ReleaserPropertiesUpdater().updateProperties(this.releaserProperties,
+ project);
+ this.projectPomUpdater.updateProjectFromReleaseTrain(project, versions, versionFromBom, assertSnapshots);
+ this.gradleUpdater.updateProjectFromReleaseTrain(updatedProperties, project, versions, versionFromBom,
+ assertSnapshots);
ProjectVersion changedVersion = new ProjectVersion(project);
log.info("\n\nProject was successfully updated to [{}]", changedVersion.version);
return ExecutionResult.success();
}
- public ExecutionResult buildProject(ReleaserProperties properties,
- ProjectVersion originalVersion, ProjectVersion versionFromBom) {
+ public ExecutionResult buildProject(ReleaserProperties properties, ProjectVersion originalVersion,
+ ProjectVersion versionFromBom) {
this.projectCommandExecutor.build(properties, originalVersion, versionFromBom);
log.info("\nProject was successfully built");
return ExecutionResult.success();
}
- public ExecutionResult commitAndPushTags(File project,
- ProjectVersion changedVersion) {
+ public ExecutionResult commitAndPushTags(File project, ProjectVersion changedVersion) {
this.projectGitHandler.commitAndTagIfApplicable(project, changedVersion);
log.info("\nCommit was made and tag was pushed successfully");
return ExecutionResult.success();
}
- public ExecutionResult deploy(ReleaserProperties properties,
- ProjectVersion originalVersion, ProjectVersion versionFromBom) {
+ public ExecutionResult deploy(ReleaserProperties properties, ProjectVersion originalVersion,
+ ProjectVersion versionFromBom) {
this.projectCommandExecutor.deploy(properties, originalVersion, versionFromBom);
log.info("\nThe artifact was deployed successfully");
return ExecutionResult.success();
}
- public ExecutionResult publishDocs(ReleaserProperties properties,
- ProjectVersion originalVersion, ProjectVersion changedVersion) {
- this.projectCommandExecutor.publishDocs(properties, originalVersion,
- changedVersion);
+ public ExecutionResult publishDocs(ReleaserProperties properties, ProjectVersion originalVersion,
+ ProjectVersion changedVersion) {
+ this.projectCommandExecutor.publishDocs(properties, originalVersion, changedVersion);
log.info("\nThe docs were published successfully");
return ExecutionResult.success();
}
- public ExecutionResult rollbackReleaseVersion(File project, Projects projects,
- ProjectVersion scReleaseVersion) {
+ public ExecutionResult rollbackReleaseVersion(File project, Projects projects, ProjectVersion scReleaseVersion) {
if (scReleaseVersion.isSnapshot()) {
log.info("\nWon't rollback a snapshot version");
return ExecutionResult.skipped();
@@ -158,34 +150,29 @@ public class Releaser {
this.projectGitHandler.revertChangesIfApplicable(project, scReleaseVersion);
ProjectVersion originalVersion = originalVersion(project);
log.info("Original project version is [{}]", originalVersion);
- if ((scReleaseVersion.isRelease() || scReleaseVersion.isServiceRelease())
- && originalVersion.isSnapshot()) {
+ if ((scReleaseVersion.isRelease() || scReleaseVersion.isServiceRelease()) && originalVersion.isSnapshot()) {
updateBumpedVersions(project, projects, originalVersion);
}
else {
- log.info(
- "\nSuccessfully reverted the commit and came back to snapshot versions");
+ log.info("\nSuccessfully reverted the commit and came back to snapshot versions");
}
return ExecutionResult.success();
}
- private ExecutionResult updateBumpedVersions(File project, Projects projects,
- ProjectVersion originalVersion) {
+ private ExecutionResult updateBumpedVersions(File project, Projects projects, ProjectVersion originalVersion) {
Projects newProjects = Projects.forRollback(releaserProperties, projects);
ProjectVersion bumpedProject = bumpProject(originalVersion, newProjects);
log.info("Will bump versions \n{}", newProjects);
- updateProjectFromBom(project, newProjects, originalVersion,
- SKIP_SNAPSHOT_ASSERTION);
+ updateProjectFromBom(project, newProjects, originalVersion, SKIP_SNAPSHOT_ASSERTION);
this.projectGitHandler.commitAfterBumpingVersions(project, bumpedProject);
log.info("\nSuccessfully reverted the commit and bumped snapshot versions");
return ExecutionResult.success();
}
- private ProjectVersion bumpProject(ProjectVersion originalVersion,
- Projects newProjects) {
+ private ProjectVersion bumpProject(ProjectVersion originalVersion, Projects newProjects) {
return newProjects.containsProject(originalVersion.projectName)
- ? newProjects.forName(originalVersion.projectName) : new ProjectVersion(
- originalVersion.projectName, originalVersion.bumpedVersion());
+ ? newProjects.forName(originalVersion.projectName)
+ : new ProjectVersion(originalVersion.projectName, originalVersion.bumpedVersion());
}
ProjectVersion originalVersion(File project) {
@@ -214,10 +201,8 @@ public class Releaser {
}
public ExecutionResult createEmail(ProjectVersion releaseVersion, Projects projects) {
- Assert.notNull(releaseVersion,
- "You must provide a release version for your project");
- Assert.notNull(releaseVersion.version,
- "You must provide a release version for your project");
+ Assert.notNull(releaseVersion, "You must provide a release version for your project");
+ Assert.notNull(releaseVersion.version, "You must provide a release version for your project");
if (releaseVersion.isSnapshot()) {
log.info("\nWon't create email template for a SNAPSHOT version");
return ExecutionResult.skipped();
@@ -229,13 +214,11 @@ public class Releaser {
return ExecutionResult.success();
}
else {
- return ExecutionResult.unstable(
- new BuildUnstableException("Failed to create an email template"));
+ return ExecutionResult.unstable(new BuildUnstableException("Failed to create an email template"));
}
}
catch (Exception ex) {
- return ExecutionResult.unstable(
- new BuildUnstableException("Failed to create an email template", ex));
+ return ExecutionResult.unstable(new BuildUnstableException("Failed to create an email template", ex));
}
}
@@ -251,53 +234,44 @@ public class Releaser {
return ExecutionResult.success();
}
else {
- return ExecutionResult.unstable(
- new BuildUnstableException("Failed to create a blog template"));
+ return ExecutionResult.unstable(new BuildUnstableException("Failed to create a blog template"));
}
}
catch (Exception ex) {
- return ExecutionResult.unstable(
- new BuildUnstableException("Failed to create a blog template", ex));
+ return ExecutionResult.unstable(new BuildUnstableException("Failed to create a blog template", ex));
}
}
- public ExecutionResult updateSpringGuides(ProjectVersion releaseVersion,
- Projects projects, List processedProjects) {
+ public ExecutionResult updateSpringGuides(ProjectVersion releaseVersion, Projects projects,
+ List processedProjects) {
if (!(releaseVersion.isRelease() || releaseVersion.isServiceRelease())) {
- log.info(
- "\nWon't update Spring Guides for a non Release / Service Release version");
+ log.info("\nWon't update Spring Guides for a non Release / Service Release version");
return ExecutionResult.skipped();
}
- Exception springGuidesException = createIssueInSpringGuides(releaseVersion,
- projects);
+ Exception springGuidesException = createIssueInSpringGuides(releaseVersion, projects);
Exception deployGuidesException = deployGuides(processedProjects);
if (springGuidesException != null || deployGuidesException != null) {
- return ExecutionResult.unstable(new BuildUnstableException(
- "Failed to update Spring Guides. Spring Guides updated successfully ["
- + (springGuidesException != null)
- + "] deployed guides successfully ["
+ return ExecutionResult.unstable(
+ new BuildUnstableException("Failed to update Spring Guides. Spring Guides updated successfully ["
+ + (springGuidesException != null) + "] deployed guides successfully ["
+ (deployGuidesException != null) + "]"));
}
return ExecutionResult.success();
}
- public ExecutionResult updateStartSpringIo(ProjectVersion releaseVersion,
- Projects projects) {
+ public ExecutionResult updateStartSpringIo(ProjectVersion releaseVersion, Projects projects) {
if (!(releaseVersion.isRelease() || releaseVersion.isServiceRelease())) {
- log.info(
- "\nWon't update start.spring.io for a non Release / Service Release version");
+ log.info("\nWon't update start.spring.io for a non Release / Service Release version");
return ExecutionResult.skipped();
}
Exception exception = createIssueInStartSpringIo(releaseVersion, projects);
if (exception != null) {
- return ExecutionResult.unstable(
- new BuildUnstableException("Failed to update start.spring.io"));
+ return ExecutionResult.unstable(new BuildUnstableException("Failed to update start.spring.io"));
}
return ExecutionResult.success();
}
- private Exception createIssueInSpringGuides(ProjectVersion releaseVersion,
- Projects projects) {
+ private Exception createIssueInSpringGuides(ProjectVersion releaseVersion, Projects projects) {
try {
this.projectGitHubHandler.createIssueInSpringGuides(projects, releaseVersion);
log.info("\nSuccessfully created an issue in Spring Guides");
@@ -309,18 +283,15 @@ public class Releaser {
}
}
- private Exception createIssueInStartSpringIo(ProjectVersion releaseVersion,
- Projects projects) {
+ private Exception createIssueInStartSpringIo(ProjectVersion releaseVersion, Projects projects) {
try {
- this.projectGitHubHandler.createIssueInStartSpringIo(projects,
- releaseVersion);
+ this.projectGitHubHandler.createIssueInStartSpringIo(projects, releaseVersion);
log.info("\nSuccessfully created an issue in start.spring.io");
return null;
}
catch (Exception ex) {
log.error("Failed to update start.spring.io repo", ex);
- return new BuildUnstableException("Failed to update start.spring.io repo",
- ex);
+ return new BuildUnstableException("Failed to update start.spring.io repo", ex);
}
}
@@ -348,18 +319,15 @@ public class Releaser {
return ExecutionResult.success();
}
else {
- return ExecutionResult.unstable(
- new BuildUnstableException("Failed to create a tweet template"));
+ return ExecutionResult.unstable(new BuildUnstableException("Failed to create a tweet template"));
}
}
catch (Exception ex) {
- return ExecutionResult.unstable(
- new BuildUnstableException("Failed to create a tweet template", ex));
+ return ExecutionResult.unstable(new BuildUnstableException("Failed to create a tweet template", ex));
}
}
- public ExecutionResult createReleaseNotes(ProjectVersion releaseVersion,
- Projects projects) {
+ public ExecutionResult createReleaseNotes(ProjectVersion releaseVersion, Projects projects) {
if (releaseVersion.isSnapshot()) {
log.info("\nWon't create release notes for a SNAPSHOT version");
return ExecutionResult.skipped();
@@ -370,18 +338,15 @@ public class Releaser {
return ExecutionResult.success();
}
catch (Exception ex) {
- return ExecutionResult.unstable(
- new BuildUnstableException("Failed to create release notes", ex));
+ return ExecutionResult.unstable(new BuildUnstableException("Failed to create release notes", ex));
}
}
- public ExecutionResult updateSagan(File project, ProjectVersion releaseVersion,
- Projects projects) {
+ public ExecutionResult updateSagan(File project, ProjectVersion releaseVersion, Projects projects) {
String currentBranch = this.projectGitHandler.currentBranch(project);
ProjectVersion originalVersion = new ProjectVersion(project);
try {
- return this.saganUpdater.updateSagan(project, currentBranch, originalVersion,
- releaseVersion, projects);
+ return this.saganUpdater.updateSagan(project, currentBranch, originalVersion, releaseVersion, projects);
}
catch (Exception ex) {
return ExecutionResult.unstable(new BuildUnstableException(ex));
diff --git a/releaser-core/src/main/java/releaser/internal/ReleaserProperties.java b/releaser-core/src/main/java/releaser/internal/ReleaserProperties.java
index 1061db9d..3dab2f5d 100644
--- a/releaser-core/src/main/java/releaser/internal/ReleaserProperties.java
+++ b/releaser-core/src/main/java/releaser/internal/ReleaserProperties.java
@@ -87,8 +87,7 @@ public class ReleaserProperties implements Serializable {
private MetaRelease metaRelease = new MetaRelease();
public String getWorkingDir() {
- return StringUtils.hasText(this.workingDir) ? this.workingDir
- : System.getProperty("user.dir");
+ return StringUtils.hasText(this.workingDir) ? this.workingDir : System.getProperty("user.dir");
}
public void setWorkingDir(String workingDir) {
@@ -201,10 +200,9 @@ public class ReleaserProperties implements Serializable {
@Override
public String toString() {
- return "ReleaserProperties{" + "workingDir='" + this.workingDir + '\'' + ", git="
- + this.git + ", pom=" + this.pom + ", maven=" + this.maven + ", gradle="
- + this.gradle + ", sagan=" + this.sagan + ", fixedVersions="
- + this.fixedVersions + ", metaRelease=" + this.metaRelease + ", template="
+ return "ReleaserProperties{" + "workingDir='" + this.workingDir + '\'' + ", git=" + this.git + ", pom="
+ + this.pom + ", maven=" + this.maven + ", gradle=" + this.gradle + ", sagan=" + this.sagan
+ + ", fixedVersions=" + this.fixedVersions + ", metaRelease=" + this.metaRelease + ", template="
+ this.template + ", versions=" + this.versions + '}';
}
@@ -362,8 +360,7 @@ public class ReleaserProperties implements Serializable {
return this.releaseTrainDependencyNames;
}
- public void setReleaseTrainDependencyNames(
- List releaseTrainDependencyNames) {
+ public void setReleaseTrainDependencyNames(List releaseTrainDependencyNames) {
this.releaseTrainDependencyNames = releaseTrainDependencyNames;
}
@@ -401,12 +398,10 @@ public class ReleaserProperties implements Serializable {
@Override
public String toString() {
- return "MetaRelease{" + "enabled=" + enabled + ", releaseTrainProjectName='"
- + releaseTrainProjectName + '\'' + ", releaseTrainDependencyNames="
- + releaseTrainDependencyNames + ", gitOrgUrl='" + gitOrgUrl + '\''
- + ", projectsToSkip=" + projectsToSkip + ", releaseGroups="
- + releaseGroups + ", releaseGroupTimeoutInMinutes="
- + releaseGroupTimeoutInMinutes + ", releaseGroupThreadCount="
+ return "MetaRelease{" + "enabled=" + enabled + ", releaseTrainProjectName='" + releaseTrainProjectName
+ + '\'' + ", releaseTrainDependencyNames=" + releaseTrainDependencyNames + ", gitOrgUrl='"
+ + gitOrgUrl + '\'' + ", projectsToSkip=" + projectsToSkip + ", releaseGroups=" + releaseGroups
+ + ", releaseGroupTimeoutInMinutes=" + releaseGroupTimeoutInMinutes + ", releaseGroupThreadCount="
+ releaseGroupThreadCount + '}';
}
@@ -823,21 +818,16 @@ public class ReleaserProperties implements Serializable {
@Override
public String toString() {
- return "Git{" + "releaseTrainBomUrl='" + this.releaseTrainBomUrl + '\''
- + ", documentationUrl='" + this.documentationUrl + '\''
- + ", documentationBranch='" + this.documentationBranch + '\''
- + ", releaseTrainBranch='" + this.releaseTrainBranch + '\''
- + ", releaseTrainWikiUrl='" + this.releaseTrainWikiUrl + '\''
- + ", updateDocumentationRepo=" + this.updateDocumentationRepo
- + ", springProjectUrl=" + this.springProjectUrl
- + ", springProjectBranch=" + this.springProjectBranch
- + ", releaseTrainWikiPagePrefix=" + this.releaseTrainWikiPagePrefix
- + ", cloneDestinationDir='" + this.cloneDestinationDir + '\''
- + ", fetchVersionsFromGit=" + this.fetchVersionsFromGit
- + ", numberOfCheckedMilestones=" + this.numberOfCheckedMilestones
- + ", updateSpringGuides=" + this.updateSpringGuides
- + ", updateSpringProject=" + this.updateSpringProject
- + ", sampleUrlsSize=" + this.allTestSampleUrls.size() + '}';
+ return "Git{" + "releaseTrainBomUrl='" + this.releaseTrainBomUrl + '\'' + ", documentationUrl='"
+ + this.documentationUrl + '\'' + ", documentationBranch='" + this.documentationBranch + '\''
+ + ", releaseTrainBranch='" + this.releaseTrainBranch + '\'' + ", releaseTrainWikiUrl='"
+ + this.releaseTrainWikiUrl + '\'' + ", updateDocumentationRepo=" + this.updateDocumentationRepo
+ + ", springProjectUrl=" + this.springProjectUrl + ", springProjectBranch="
+ + this.springProjectBranch + ", releaseTrainWikiPagePrefix=" + this.releaseTrainWikiPagePrefix
+ + ", cloneDestinationDir='" + this.cloneDestinationDir + '\'' + ", fetchVersionsFromGit="
+ + this.fetchVersionsFromGit + ", numberOfCheckedMilestones=" + this.numberOfCheckedMilestones
+ + ", updateSpringGuides=" + this.updateSpringGuides + ", updateSpringProject="
+ + this.updateSpringProject + ", sampleUrlsSize=" + this.allTestSampleUrls.size() + '}';
}
}
@@ -918,11 +908,10 @@ public class ReleaserProperties implements Serializable {
@Override
public String toString() {
- return "Pom{" + "branch='" + this.branch + '\''
- + ", pomWithBootStarterParent='" + this.pomWithBootStarterParent
- + '\'' + ", thisTrainBom='" + this.thisTrainBom + '\''
- + ", bomVersionPattern='" + this.bomVersionPattern + '\''
- + ", ignoredPomRegex=" + this.ignoredPomRegex + '}';
+ return "Pom{" + "branch='" + this.branch + '\'' + ", pomWithBootStarterParent='"
+ + this.pomWithBootStarterParent + '\'' + ", thisTrainBom='" + this.thisTrainBom + '\''
+ + ", bomVersionPattern='" + this.bomVersionPattern + '\'' + ", ignoredPomRegex="
+ + this.ignoredPomRegex + '}';
}
}
@@ -1039,8 +1028,7 @@ public class ReleaserProperties implements Serializable {
}
@Override
- public void setGenerateReleaseTrainDocsCommand(
- String generateReleaseTrainDocsCommand) {
+ public void setGenerateReleaseTrainDocsCommand(String generateReleaseTrainDocsCommand) {
this.generateReleaseTrainDocsCommand = generateReleaseTrainDocsCommand;
}
@@ -1056,12 +1044,10 @@ public class ReleaserProperties implements Serializable {
@Override
public String toString() {
- return "Maven{" + "buildCommand='" + this.buildCommand + '\''
- + ", deployCommand='" + this.deployCommand + '\''
- + ", publishDocsCommand=" + this.publishDocsCommand
- + "generateReleaseTrainDocsCommand='"
- + this.generateReleaseTrainDocsCommand + '\'' + ", waitTimeInMinutes="
- + this.waitTimeInMinutes + '}';
+ return "Maven{" + "buildCommand='" + this.buildCommand + '\'' + ", deployCommand='" + this.deployCommand
+ + '\'' + ", publishDocsCommand=" + this.publishDocsCommand + "generateReleaseTrainDocsCommand='"
+ + this.generateReleaseTrainDocsCommand + '\'' + ", waitTimeInMinutes=" + this.waitTimeInMinutes
+ + '}';
}
}
@@ -1173,8 +1159,7 @@ public class ReleaserProperties implements Serializable {
}
@Override
- public void setGenerateReleaseTrainDocsCommand(
- String generateReleaseTrainDocsCommand) {
+ public void setGenerateReleaseTrainDocsCommand(String generateReleaseTrainDocsCommand) {
this.generateReleaseTrainDocsCommand = generateReleaseTrainDocsCommand;
}
@@ -1190,12 +1175,10 @@ public class ReleaserProperties implements Serializable {
@Override
public String toString() {
- return "Bash{" + "buildCommand='" + this.buildCommand + '\''
- + ", deployCommand='" + this.deployCommand + '\''
- + ", publishDocsCommand=" + this.publishDocsCommand
- + "generateReleaseTrainDocsCommand='"
- + this.generateReleaseTrainDocsCommand + '\'' + ", waitTimeInMinutes="
- + this.waitTimeInMinutes + '}';
+ return "Bash{" + "buildCommand='" + this.buildCommand + '\'' + ", deployCommand='" + this.deployCommand
+ + '\'' + ", publishDocsCommand=" + this.publishDocsCommand + "generateReleaseTrainDocsCommand='"
+ + this.generateReleaseTrainDocsCommand + '\'' + ", waitTimeInMinutes=" + this.waitTimeInMinutes
+ + '}';
}
}
@@ -1328,8 +1311,7 @@ public class ReleaserProperties implements Serializable {
}
@Override
- public void setGenerateReleaseTrainDocsCommand(
- String generateReleaseTrainDocsCommand) {
+ public void setGenerateReleaseTrainDocsCommand(String generateReleaseTrainDocsCommand) {
this.generateReleaseTrainDocsCommand = generateReleaseTrainDocsCommand;
}
@@ -1347,8 +1329,7 @@ public class ReleaserProperties implements Serializable {
return this.gradlePropsSubstitution;
}
- public void setGradlePropsSubstitution(
- Map gradlePropsSubstitution) {
+ public void setGradlePropsSubstitution(Map gradlePropsSubstitution) {
this.gradlePropsSubstitution = gradlePropsSubstitution;
}
@@ -1364,15 +1345,13 @@ public class ReleaserProperties implements Serializable {
public String toString() {
return new StringJoiner(", ", Gradle.class.getSimpleName() + "[", "]")
.add("gradlePropsSubstitution=" + gradlePropsSubstitution)
- .add("ignoredGradleRegex=" + ignoredGradleRegex)
- .add("buildCommand='" + buildCommand + "'")
+ .add("ignoredGradleRegex=" + ignoredGradleRegex).add("buildCommand='" + buildCommand + "'")
.add("deployCommand='" + deployCommand + "'")
.add("deployGuidesCommand='" + deployGuidesCommand + "'")
.add("publishDocsCommand=" + publishDocsCommand)
- .add("generateReleaseTrainDocsCommand='"
- + generateReleaseTrainDocsCommand + "'")
- .add("systemProperties='" + systemProperties + "'")
- .add("waitTimeInMinutes=" + waitTimeInMinutes).toString();
+ .add("generateReleaseTrainDocsCommand='" + generateReleaseTrainDocsCommand + "'")
+ .add("systemProperties='" + systemProperties + "'").add("waitTimeInMinutes=" + waitTimeInMinutes)
+ .toString();
}
}
@@ -1519,8 +1498,8 @@ public class ReleaserProperties implements Serializable {
@Override
public String toString() {
- return "Versions{" + "allVersionsFileUrl='" + this.allVersionsFileUrl + '\''
- + ", bomName='" + this.bomName + '\'' + '}';
+ return "Versions{" + "allVersionsFileUrl='" + this.allVersionsFileUrl + '\'' + ", bomName='" + this.bomName
+ + '\'' + '}';
}
}
diff --git a/releaser-core/src/main/java/releaser/internal/ReleaserPropertiesUpdater.java b/releaser-core/src/main/java/releaser/internal/ReleaserPropertiesUpdater.java
index d8633992..837fe4dc 100644
--- a/releaser-core/src/main/java/releaser/internal/ReleaserPropertiesUpdater.java
+++ b/releaser-core/src/main/java/releaser/internal/ReleaserPropertiesUpdater.java
@@ -41,13 +41,11 @@ import org.springframework.util.StringUtils;
*/
public class ReleaserPropertiesUpdater implements Closeable {
- private static final Logger log = LoggerFactory
- .getLogger(ReleaserPropertiesUpdater.class);
+ private static final Logger log = LoggerFactory.getLogger(ReleaserPropertiesUpdater.class);
private static final Map CACHE = new ConcurrentHashMap<>();
- public ReleaserProperties updateProperties(ReleaserProperties properties,
- File clonedProjectFromOrg) {
+ public ReleaserProperties updateProperties(ReleaserProperties properties, File clonedProjectFromOrg) {
return CACHE.computeIfAbsent(clonedProjectFromOrg, file -> {
ReleaserProperties props = updatePropertiesFromFile(properties.copy(), file);
props.setWorkingDir(clonedProjectFromOrg.getAbsolutePath());
@@ -56,8 +54,7 @@ public class ReleaserPropertiesUpdater implements Closeable {
});
}
- private ReleaserProperties updatePropertiesFromFile(ReleaserProperties copy,
- File clonedProjectFromOrg) {
+ private ReleaserProperties updatePropertiesFromFile(ReleaserProperties copy, File clonedProjectFromOrg) {
File releaserConfig = releaserConfig(clonedProjectFromOrg);
if (releaserConfig.exists()) {
try {
@@ -66,11 +63,8 @@ public class ReleaserPropertiesUpdater implements Closeable {
Properties properties = yamlProcessor.getObject();
ReleaserProperties releaserProperties = new Binder(
new MapConfigurationPropertySource(properties.entrySet().stream()
- .collect(Collectors.toMap(e -> e.getKey().toString(),
- e -> e.getValue().toString()))))
- .bind("releaser",
- ReleaserProperties.class)
- .get();
+ .collect(Collectors.toMap(e -> e.getKey().toString(), e -> e.getValue().toString()))))
+ .bind("releaser", ReleaserProperties.class).get();
log.info("config/releaser.yml found. Will update the current properties");
overrideProperties(releaserProperties, copy);
}
@@ -79,17 +73,14 @@ public class ReleaserPropertiesUpdater implements Closeable {
}
}
else {
- log.info(
- "No config/releaser.yml found. Will NOT update the current properties");
+ log.info("No config/releaser.yml found. Will NOT update the current properties");
}
- log.info("Updating working directory to [{}]",
- clonedProjectFromOrg.getAbsolutePath());
+ log.info("Updating working directory to [{}]", clonedProjectFromOrg.getAbsolutePath());
copy.setWorkingDir(clonedProjectFromOrg.getAbsolutePath());
return copy;
}
- private void overrideProperties(ReleaserProperties fromProject,
- ReleaserProperties copy) {
+ private void overrideProperties(ReleaserProperties fromProject, ReleaserProperties copy) {
overrideCommandIfPresent(fromProject.getMaven(), copy.getMaven());
overrideCommandIfPresent(fromProject.getGradle(), copy.getGradle());
overrideMapIfPresent(() -> fromProject.getGradle().getGradlePropsSubstitution(),
@@ -101,45 +92,36 @@ public class ReleaserPropertiesUpdater implements Closeable {
private void overrideCommandIfPresent(ReleaserProperties.Command fromProject,
ReleaserProperties.Command commandCopy) {
- overrideStringIfPresent(fromProject::getBuildCommand,
- commandCopy::setBuildCommand);
- overrideStringIfPresent(fromProject::getDeployCommand,
- commandCopy::setDeployCommand);
+ overrideStringIfPresent(fromProject::getBuildCommand, commandCopy::setBuildCommand);
+ overrideStringIfPresent(fromProject::getDeployCommand, commandCopy::setDeployCommand);
overrideStringIfPresent(fromProject::getGenerateReleaseTrainDocsCommand,
commandCopy::setGenerateReleaseTrainDocsCommand);
- overrideStringIfPresent(fromProject::getPublishDocsCommand,
- commandCopy::setPublishDocsCommand);
- overrideStringIfPresent(fromProject::getDeployGuidesCommand,
- commandCopy::setDeployGuidesCommand);
- overrideStringIfPresent(fromProject::getSystemProperties,
- commandCopy::setSystemProperties);
+ overrideStringIfPresent(fromProject::getPublishDocsCommand, commandCopy::setPublishDocsCommand);
+ overrideStringIfPresent(fromProject::getDeployGuidesCommand, commandCopy::setDeployGuidesCommand);
+ overrideStringIfPresent(fromProject::getSystemProperties, commandCopy::setSystemProperties);
}
- private void overrideStringIfPresent(Supplier predicate,
- Consumer consumer) {
+ private void overrideStringIfPresent(Supplier predicate, Consumer consumer) {
if (StringUtils.hasText(predicate.get())) {
consumer.accept(predicate.get());
}
}
- private void overrideArrayIfPresent(Supplier predicate,
- Consumer consumer) {
+ private void overrideArrayIfPresent(Supplier predicate, Consumer consumer) {
String[] strings = predicate.get();
if (strings.length > 0) {
consumer.accept(strings);
}
}
- private void overrideMapIfPresent(Supplier