-
Notifications
You must be signed in to change notification settings - Fork 654
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
Add GitHub actions for SDK and codegen CI testing #1261
Conversation
cac3086
to
b0b4b6d
Compare
Looks like test are overall faster than Travis, also codegen java 11 failed on Travis. We should consider dropping all Travis tests except Go tip tests. |
63bd82e
to
392857f
Compare
Adds GitHub Actions for Go 1.15, 1.16 SDK unit testing and Java codegen. Go tip is not included yet, relying on to rely on Travis CI for these until added to GitHub Actions.
- uses: actions/setup-go@v2 | ||
with: | ||
go-version: ${{ matrix.go-version }} |
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.
This is much nicer then what I had to do with Travis CI recently.
# os: [windows-latest] | ||
# experimental: true | ||
env: | ||
EACHMODULE_SKIP: "internal\\repotools\\changes" |
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.
We can probably remove this code now, we can do that in a separate PR.
Adds GitHub Actions for Go 1.15, 1.16 SDK unit testing and Java codegen. Go tip is not included yet, relying on to rely on Travis CI for these until added to GitHub Actions.
Adds GitHub Actions for Go 1.15, 1.16 SDK unit testing and Java codegen. Go tip is not included yet, relying on to rely on Travis CI for these until added to GitHub Actions.