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
TCN-875 Various bugs in release action #86
TCN-875 Various bugs in release action #86
Conversation
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 might not be the best person to review this one 😄; is this referencing release workflows we have elsewhere?
commit-message: "Update version to v${{env.VERSION}}" | ||
branch: release/v${{env.VERSION}} | ||
delete-branch: true | ||
title: "Release ${{env.VERSION}}" | ||
body: Release prepared for ${{env.VERSION}} | ||
title: "Release v${{env.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.
won't these v
s be doubled? (I realize this looks a lot like https://github.com/bufbuild/buf/blob/main/.github/workflows/release.yaml so maybe not?)
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.
these will not, the input should be x.y.z without any v, even though that is not enforced at this stage 😂
private_key: ${{ secrets.TOKEN_EXCHANGE_GH_APP_PRIVATE_KEY }} | ||
repository: ${{ github.repository }} | ||
permissions: >- | ||
{"contents": "write"} |
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.
does this need "pull_requests": "write"
permission, similar to buf
's release workflow?
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 one does not, the one you reference is OP
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.
stamping assuming you know what you're doing 😄
v
in pr outputv1
branch never completedall resolved.