Skip to content

Commit 22b7a55

Browse files
committed
chore(codegen): return ArrayNode from partition.getVariants()
1 parent e0aa67a commit 22b7a55

File tree

1 file changed

+3
-7
lines changed
  • codegen/smithy-aws-typescript-codegen/src/main/java/software/amazon/smithy/aws/typescript/codegen

1 file changed

+3
-7
lines changed

codegen/smithy-aws-typescript-codegen/src/main/java/software/amazon/smithy/aws/typescript/codegen/EndpointGenerator.java

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -145,11 +145,7 @@ private void writePartitionHash() {
145145

146146
// TODO: Remove hostname after fully switching to variants.
147147
writer.write("hostname: $S,", partition.hostnameTemplate);
148-
writer.openBlock("variants: [", "],", () -> {
149-
partition.getVariants().forEach(variant -> {
150-
writer.write("$L, ", Node.prettyPrintJson(variant));
151-
});
152-
});
148+
writer.write("variants: $L,", ArrayNode.prettyPrintJson(partition.getVariants()));
153149

154150
partition.getPartitionEndpoint().ifPresent(
155151
endpoint -> writer.write("endpoint: $S,", endpoint));
@@ -251,7 +247,7 @@ Set<String> getAllRegions() {
251247
return regions;
252248
}
253249

254-
List<Node> getVariants() {
250+
ArrayNode getVariants() {
255251
List<Node> allVariants = new ArrayList<Node>();
256252

257253
if (defaults.containsMember("variants")) {
@@ -270,7 +266,7 @@ List<Node> getVariants() {
270266
}
271267
allVariants.add(getDefaultVariant(hostnameTemplate));
272268

273-
return allVariants;
269+
return ArrayNode.fromNodes(allVariants);
274270
}
275271

276272
Optional<String> getPartitionEndpoint() {

0 commit comments

Comments
 (0)