-
Notifications
You must be signed in to change notification settings - Fork 10
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
Support for building gems and prepare release #171
Conversation
uses: arduino/setup-protoc@v3 | ||
with: | ||
# TODO(cretz): Can upgrade proto when https://github.com/arduino/setup-protoc/issues/99 fixed | ||
version: "23.x" | ||
repo-token: ${{ secrets.GITHUB_TOKEN }} | ||
|
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.
To other reviewers: this has been moved to temporalio/Rakefile
working-directory: ./temporalio | ||
cache-version: v1-${{ matrix.rubyPlatform }} | ||
|
||
- name: Cross compile gems |
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 suppose that it will output files to temporalio/internal/bridge/3.[123]/temporalio_bridge.(extension)
. Is that exact?
Some comments on what this action will do would be useful to future readers.
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.
Yes. Not really wanting to re-document what a separate action does here where we use it, but the internal/bridge.rb
does have version specific loading
Can you please brush up a bit what you envision to be the release procedure? I don't think we need to formally include that in the repo just now, that can certainly wait until we get experience from doing a few releases. But having some idea of where you are heading to would help us in validating this PR. |
Yes, this will be documented internally alongside the release process for the other SDKs. Basically it's just downloading the gems artifacts from the build-gems GH workflow run and using |
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 good to me, thanks!
What was changed