Skip to content
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

test rebase from smithy-codegen to master (with cherry-pick for commit with conflict issues) #4

Closed
wants to merge 586 commits into from

Conversation

trivikr
Copy link
Owner

@trivikr trivikr commented Mar 20, 2020

No description provided.

Chase Coalwell and others added 30 commits January 9, 2020 16:40
* feat: infer endpoint prefix from a map file

* fix: update copy script to override endpoints.ts

* chore: add endpoint functional test cases

* fix: update clients with correct endpoint prefix from C2J
This commit adds support for the `aws.rest-xml` protocol, building
on top of the `HttpBindingProtocolGenerator` for document and
payload serde.

Implementations of the `DocumentMember[Deser|Ser]Visitor` and the
`DocumentShape[Deser|Ser]Visitor` have been created that handle
Smithy's XML traits and their influence on protocol serde.

A minor update has been made to the `XmlNode` to allow for nodes to
be renamed, as the same structure may change XML node names when it
is bound to different locations.
AllanZhengYP and others added 23 commits March 6, 2020 09:42
* Add support for generating HTTP protocol tests(awslabls/smithy-typescript@6b6a3b182771e00742369e949903087378862ea2)

* chore: update yarn lock
The @aws-crypto/sha256-browser package doesn't work properly in ReactNative. So we use replace it will pure JS hasher(@aws-crypto/sha256-js) instead.
Codegen for smithy-lang/smithy-typescript#144
* fix(smithy-client): string.toString requires this be a String

Because of Because of microsoft/tslib#95,
TS 'extends' shim doesn't support extending native types like String.
So here we create StringWrapper that duplicate everything from String
class including its prototype chain. So we can extend from here.

* fix: typo
@trivikr
Copy link
Owner Author

trivikr commented Mar 20, 2020

Although the commit with resolve conflicts was cherry-picked in ffe7134, this PR shows conflicts

Screen Shot 2020-03-20 at 1 57 16 PM

@trivikr
Copy link
Owner Author

trivikr commented Mar 20, 2020

Closing as merge commit is created again, blocking "Rebase and merge"
Screen Shot 2020-03-20 at 1 58 47 PM

@trivikr trivikr force-pushed the master branch 2 times, most recently from 17e85a9 to 4a23bb4 Compare March 20, 2020 21:07
@trivikr trivikr closed this Mar 20, 2020
@trivikr trivikr deleted the smithy-codegen-pr-test-2 branch June 16, 2020 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants