-
Notifications
You must be signed in to change notification settings - Fork 37
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
Enable SDK-Java/Python/Golang integration tests running #1786
Conversation
9a5e4be
to
7d17f1f
Compare
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.
Thanks for integrating the integrations tests with the runtime. The changes look good to me. +1 for merging.
sdk-java: | ||
name: Run SDK-Java integration tests | ||
permissions: | ||
contents: read | ||
issues: read | ||
checks: write | ||
pull-requests: write | ||
actions: read | ||
secrets: inherit | ||
needs: docker | ||
uses: restatedev/sdk-java/.github/workflows/integration.yaml@main | ||
with: | ||
restateCommit: ${{ github.event.pull_request.head.sha || github.sha }} | ||
|
||
sdk-python: | ||
name: Run SDK-Python integration tests | ||
permissions: | ||
contents: read | ||
issues: read | ||
checks: write | ||
pull-requests: write | ||
actions: read | ||
secrets: inherit | ||
needs: docker | ||
uses: restatedev/sdk-python/.github/workflows/integration.yaml@main | ||
with: | ||
restateCommit: ${{ github.event.pull_request.head.sha || github.sha }} | ||
|
||
sdk-go: | ||
name: Run SDK-Go integration tests | ||
permissions: | ||
contents: read | ||
issues: read | ||
checks: write | ||
pull-requests: write | ||
actions: read | ||
secrets: inherit | ||
needs: docker | ||
uses: restatedev/sdk-go/.github/workflows/integration.yaml@main | ||
with: | ||
restateCommit: ${{ github.event.pull_request.head.sha || github.sha }} |
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.
I guess in the future, we could only run the integration tests for a selected SDK and have the other SDKs tested against main on a scheduled basis if the number of Github action minutes becomes too large.
No description provided.