remove VersionScmPosition class as it is a duplicate of ScmPosition c…

…lass (#686)
allegro · Dec 4, 2023 · 5472530 · 5472530
1 parent 9eee1c8
commit 5472530
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 33 deletions.
diff --git a/src/main/groovy/pl/allegro/tech/build/axion/release/domain/VersionConfig.groovy b/src/main/groovy/pl/allegro/tech/build/axion/release/domain/VersionConfig.groovy
@@ -11,6 +11,7 @@ import org.gradle.api.tasks.Nested
 import org.gradle.api.tasks.Optional
 import pl.allegro.tech.build.axion.release.domain.hooks.HooksConfig
 import pl.allegro.tech.build.axion.release.domain.properties.VersionProperties
+import pl.allegro.tech.build.axion.release.domain.scm.ScmPosition
 import pl.allegro.tech.build.axion.release.infrastructure.di.MemoizedVersionSupplier
 import pl.allegro.tech.build.axion.release.infrastructure.di.VersionSupplier
 
@@ -239,11 +240,7 @@ abstract class VersionConfig extends BaseExtension {
     }
 
     @Nested
-    VersionScmPosition getScmPosition() {
-        return versionProvider().map({ new VersionScmPosition(
-            it.position.revision,
-            it.position.shortRevision,
-            it.position.branch
-        )}).get()
+    ScmPosition getScmPosition() {
+        return versionProvider().map({it.position}).get()
     }
 }
diff --git a/src/main/groovy/pl/allegro/tech/build/axion/release/domain/VersionScmPosition.groovy b/src/main/groovy/pl/allegro/tech/build/axion/release/domain/VersionScmPosition.groovy