-
Notifications
You must be signed in to change notification settings - Fork 105
Use Array shorthand - Type[] instead of Array<Type> #168
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
CodeGen PR: aws/aws-sdk-js-v3#1127 |
...ypescript-codegen/src/main/java/software/amazon/smithy/typescript/codegen/SymbolVisitor.java
Show resolved
Hide resolved
...in/java/software/amazon/smithy/typescript/codegen/integration/DocumentShapeDeserVisitor.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like there's a few tests that don't enforce the <>
syntax, but use it when building test symbols. Should probably update those too for clarity.
Which are these specific tests? Code below: |
Updated tests which doesn't enforce |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good from the Java side.
Merging as Allan has already approved CodeGen on JavaScript side in aws/aws-sdk-js-v3#1127 (review) |
Issue #, if available:
Refs: aws/aws-sdk-js-v3#1126
Description of changes:
Use Array shorthand -
Type[]
instead ofArray<Type>
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.