Skip to content

Commit ccd60c4

Browse files
authored
chore: Sort dependencies before rendering into package manifest (#847)
1 parent 0d4d3ea commit ccd60c4

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed

smithy-swift-codegen/src/main/kotlin/software/amazon/smithy/swift/codegen/PackageManifestGenerator.kt

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -34,16 +34,20 @@ class PackageManifestGenerator(val ctx: ProtocolGenerator.GenerationContext) {
3434
it.getProperty("url", String::class.java).getOrNull() != null ||
3535
it.getProperty("scope", String::class.java).getOrNull() != null
3636
}
37-
val dependenciesByURL = externalDependencies.distinctBy {
38-
it.getProperty("url", String::class.java).getOrNull()
39-
?: "${it.getProperty("scope", String::class.java).get()}.${it.packageName}"
40-
}
37+
val dependenciesByURL = externalDependencies
38+
.distinctBy {
39+
it.getProperty("url", String::class.java).getOrNull()
40+
?: "${it.getProperty("scope", String::class.java).get()}.${it.packageName}"
41+
}
42+
.sorted()
4143

4244
writer.openBlock("dependencies: [", "],") {
4345
dependenciesByURL.forEach { writePackageDependency(writer, it) }
4446
}
4547

46-
val dependenciesByTarget = externalDependencies.distinctBy { it.expectProperty("target", String::class.java) + it.packageName }
48+
val dependenciesByTarget = externalDependencies
49+
.distinctBy { it.expectProperty("target", String::class.java) + it.packageName }
50+
.sorted()
4751

4852
writer.openBlock("targets: [", "]") {
4953
writer.openBlock(".target(", "),") {

0 commit comments

Comments
 (0)