Skip to content

Commit 7f80d53

Browse files
committed
Cleanup TypeScriptCodegenContext
1 parent d3f103d commit 7f80d53

File tree

2 files changed

+1
-18
lines changed

2 files changed

+1
-18
lines changed

smithy-typescript-codegen/src/main/java/software/amazon/smithy/typescript/codegen/DirectedTypeScriptCodegen.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,6 @@ public TypeScriptCodegenContext createContext(CreateContextDirective<TypeScriptS
9999
? ApplicationProtocol.createDefaultHttpApplicationProtocol()
100100
: protocolGenerator.getApplicationProtocol();
101101

102-
// TODO: consider taking directive in the constructor and have all the logic there, instead of builder
103102
return TypeScriptCodegenContext.builder()
104103
.model(directive.model())
105104
.settings(directive.settings())

smithy-typescript-codegen/src/main/java/software/amazon/smithy/typescript/codegen/TypeScriptCodegenContext.java

Lines changed: 1 addition & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -26,14 +26,12 @@
2626
import software.amazon.smithy.typescript.codegen.integration.TypeScriptIntegration;
2727
import software.amazon.smithy.utils.SmithyBuilder;
2828
import software.amazon.smithy.utils.SmithyUnstableApi;
29-
import software.amazon.smithy.utils.ToSmithyBuilder;
3029

3130
/**
3231
* Holds context related to code generation.
3332
*/
3433
@SmithyUnstableApi
35-
public final class TypeScriptCodegenContext
36-
implements CodegenContext<TypeScriptSettings, TypeScriptWriter>, ToSmithyBuilder<TypeScriptCodegenContext> {
34+
public final class TypeScriptCodegenContext implements CodegenContext<TypeScriptSettings, TypeScriptWriter> {
3735

3836
private final Model model;
3937
private final TypeScriptSettings settings;
@@ -99,20 +97,6 @@ public ApplicationProtocol applicationProtocol() {
9997
return applicationProtocol;
10098
}
10199

102-
@Override
103-
public SmithyBuilder<TypeScriptCodegenContext> toBuilder() {
104-
return builder()
105-
.model(model)
106-
.settings(settings)
107-
.symbolProvider(symbolProvider)
108-
.fileManifest(fileManifest)
109-
.writerDelegator(writerDelegator)
110-
.integrations(integrations)
111-
.runtimePlugins(runtimePlugins)
112-
.protocolGenerator(protocolGenerator)
113-
.applicationProtocol(applicationProtocol);
114-
}
115-
116100
/**
117101
* @return Returns a builder.
118102
*/

0 commit comments

Comments
 (0)