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.

Sign up for GitHub

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

Fix REST-XML Codegen Bug, Move metrics to User-Agent header, Update SDKVersion to refer to goModuleVersion #1257

Merged
merged 8 commits into from
May 13, 2021
Merged

Fix REST-XML Codegen Bug, Move metrics to User-Agent header, Update SDKVersion to refer to goModuleVersion #1257

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
bb253cf
Fix deserialization of enum types marked with payload trait
skmcgrail May 13, 2021
c0c46fc
Add serviceId and module version to user-agent header
skmcgrail May 13, 2021
4e761e0
Updated middleware to temporarily direct client metadata to user-agent
skmcgrail May 13, 2021
281e1be
internal/repotools: Support specifying alternative package location f…
skmcgrail May 13, 2021
f55eca3
Regenerated Clients
skmcgrail May 13, 2021
08fb529
Add Changelog Annotation
skmcgrail May 13, 2021
acc90ca
aws: Updated SDKVersion to refer to goModuleVersion constant.
skmcgrail May 13, 2021
b73dec4
Make go1.16 available in Travis Java builds
skmcgrail May 13, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 10 additions & 0 deletions .travis.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,11 @@ matrix:
env:
- JAVA_TOOL_OPTIONS="-Xmx1g"
before_script:
- sudo apt-get -y install golang
- go get golang.org/dl/go1.16.4
- $(go env GOPATH)/bin/go1.16.4 download
- export PATH=$HOME/sdk/go1.16.4/bin:$PATH
- export GOROOT=$HOME/sdk/go1.16.4
Comment on lines +51 to +55
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we also add this for go tip - go get golang.org/dl/gotip ?

Just to be aware ahead of time.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah we can use this strategy to replace our current gotip build process etc.

- rm -rf /tmp/smithy-go
- git clone https://github.com/aws/smithy-go /tmp/smithy-go
- pushd /tmp/smithy-go/codegen
Expand All @@ -61,6 +66,11 @@ matrix:
env:
- JAVA_TOOL_OPTIONS="-Xmx1g"
before_script:
- sudo apt-get -y install golang
- go get golang.org/dl/go1.16.4
- $(go env GOPATH)/bin/go1.16.4 download
- export PATH=$HOME/sdk/go1.16.4/bin:$PATH
- export GOROOT=$HOME/sdk/go1.16.4
- rm -rf /tmp/smithy-go
- git clone https://github.com/aws/smithy-go /tmp/smithy-go
- pushd /tmp/smithy-go/codegen
Expand Down