Skip to content

Commit 90746ba

Browse files
authored
Remove unused local variables (#245)
* Remove unused local variables * Convert method to not return anything --------- Co-authored-by: Piotrek Żygieło <pzygielo@users.noreply.github.com>
1 parent 87f44ab commit 90746ba

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -294,7 +294,7 @@ private void transformDocument(
294294

295295
Properties properties = modelTarget.getProperties();
296296

297-
String parentVersion = rewriteParent(project, modelTarget, result, releaseDescriptor, simulate);
297+
rewriteParent(project, modelTarget, result, releaseDescriptor, simulate);
298298

299299
String projectId = ArtifactUtils.versionlessKey(project.getGroupId(), project.getArtifactId());
300300

@@ -485,18 +485,17 @@ public static boolean isCiFriendlyVersion(String version) {
485485
return CI_FRIENDLY_PROPERTIES.contains(extractPropertyFromExpression(version));
486486
}
487487

488-
private String rewriteParent(
488+
private void rewriteParent(
489489
MavenProject project,
490490
Model targetModel,
491491
ReleaseResult result,
492492
ReleaseDescriptor releaseDescriptor,
493493
boolean simulate)
494494
throws ReleaseFailureException {
495-
String parentVersion = null;
496495
if (project.hasParent()) {
497496
MavenProject parent = project.getParent();
498497
String key = ArtifactUtils.versionlessKey(parent.getGroupId(), parent.getArtifactId());
499-
parentVersion = getNextVersion(releaseDescriptor, key);
498+
String parentVersion = getNextVersion(releaseDescriptor, key);
500499
if (parentVersion == null) {
501500
// MRELEASE-317
502501
parentVersion = getResolvedSnapshotVersion(key, releaseDescriptor);
@@ -516,7 +515,6 @@ private String rewriteParent(
516515
}
517516
}
518517
}
519-
return parentVersion;
520518
}
521519

522520
private void rewriteArtifactVersions(

maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/GenerateReleasePomsPhase.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -230,7 +230,6 @@ private Model createReleaseModel(
230230
releaseProject.getBuild().setPluginManagement(null);
231231

232232
// update project version
233-
String projectVersion = releaseModel.getVersion();
234233
String releaseVersion = getNextVersion(releaseDescriptor, project.getGroupId(), project.getArtifactId());
235234
releaseModel.setVersion(releaseVersion);
236235

0 commit comments

Comments
 (0)