refactor(gleam): Remove redundant vcsProcessed assignments

maennchen · sschuberth · commit 308b337e8878 · 2026-01-08T09:30:08.000+01:00
The Package data class already normalizes the VCS info by default, so
explicitly setting vcsProcessed to vcs.normalize() or VcsInfo.EMPTY is
unnecessary.

Signed-off-by: Jonatan Männchen &lt;jonatan@maennchen.ch&gt;
diff --git a/plugins/package-managers/gleam/src/main/kotlin/ManifestPackageInfo.kt b/plugins/package-managers/gleam/src/main/kotlin/ManifestPackageInfo.kt
@@ -90,7 +90,6 @@ internal data class ManifestPackageInfo(private val pkg: GleamManifest.Package)
                 )
             } ?: RemoteArtifact.EMPTY,
             vcs = vcs,
-            vcsProcessed = vcs.normalize(),
             sourceCodeOrigins = listOf(SourceCodeOrigin.ARTIFACT, SourceCodeOrigin.VCS)
         )
     }
@@ -110,7 +109,6 @@ internal data class ManifestPackageInfo(private val pkg: GleamManifest.Package)
             binaryArtifact = RemoteArtifact.EMPTY,
             sourceArtifact = RemoteArtifact.EMPTY,
             vcs = vcs,
-            vcsProcessed = vcs.normalize(),
             sourceCodeOrigins = listOf(SourceCodeOrigin.VCS)
         )
     }
@@ -128,7 +126,6 @@ internal data class ManifestPackageInfo(private val pkg: GleamManifest.Package)
             binaryArtifact = RemoteArtifact.EMPTY,
             sourceArtifact = RemoteArtifact.EMPTY,
             vcs = VcsInfo.EMPTY,
-            vcsProcessed = VcsInfo.EMPTY,
             sourceCodeOrigins = listOf(SourceCodeOrigin.VCS)
         )
 

Original file line number	Diff line number	Diff line change
`@@ -90,7 +90,6 @@ internal data class ManifestPackageInfo(private val pkg: GleamManifest.Package)`
`90`	`90`	`)`
`91`	`91`	`} ?: RemoteArtifact.EMPTY,`
`92`	`92`	`vcs = vcs,`
`93`		`- vcsProcessed = vcs.normalize(),`
`94`	`93`	`sourceCodeOrigins = listOf(SourceCodeOrigin.ARTIFACT, SourceCodeOrigin.VCS)`
`95`	`94`	`)`
`96`	`95`	`}`
`@@ -110,7 +109,6 @@ internal data class ManifestPackageInfo(private val pkg: GleamManifest.Package)`
`110`	`109`	`binaryArtifact = RemoteArtifact.EMPTY,`
`111`	`110`	`sourceArtifact = RemoteArtifact.EMPTY,`
`112`	`111`	`vcs = vcs,`
`113`		`- vcsProcessed = vcs.normalize(),`
`114`	`112`	`sourceCodeOrigins = listOf(SourceCodeOrigin.VCS)`
`115`	`113`	`)`
`116`	`114`	`}`
`@@ -128,7 +126,6 @@ internal data class ManifestPackageInfo(private val pkg: GleamManifest.Package)`
`128`	`126`	`binaryArtifact = RemoteArtifact.EMPTY,`
`129`	`127`	`sourceArtifact = RemoteArtifact.EMPTY,`
`130`	`128`	`vcs = VcsInfo.EMPTY,`
`131`		`- vcsProcessed = VcsInfo.EMPTY,`
`132`	`129`	`sourceCodeOrigins = listOf(SourceCodeOrigin.VCS)`
`133`	`130`	`)`
`134`	`131`