-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat: add bump and tag version workflow #11
base: main
Are you sure you want to change the base?
Conversation
But a new tag will be already created with #13 right? |
No, #13 only create a github release, this pr bump version and creates the tag |
I see 😄 then better to keep everything in a single action, don't see when we will use these separately, would be nice to have a single action that can take care of everything, maybe we can use an input field to find out if it is minor update or major update |
.github/workflows/bump-version.yml
Outdated
- name: Automated Version Bump | ||
id: version-bump | ||
uses: phips28/gh-action-bump-version@master | ||
with: | ||
tag-prefix: ${{ inputs.tag-prefix }} | ||
env: | ||
GITHUB_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.
This need GITHUB token? better to use https://github.com/snapshot-labs/snapshot.js/blob/master/.github/workflows/update-minor-version.yml#L17 easy to understand
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 just open a PR. The point of this PR is to do everything all a once, at the same time.
Single action is #14 |
|
||
jobs: | ||
bump-version: | ||
name: Bump Version on master |
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 action will have permission to merge to master.? 🤔
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.
It works on regular branch. But just tested, and it does not work on protected branch, it requires a Personal Access Token
, instead of 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.
utACK
These actions do not seem to work well with protected branch. Maybe explore a more all in one solution such as release it |
Add a new workflow which will:
Fix #9