Skip to content
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

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

Conversation

wa0x6e
Copy link
Contributor

@wa0x6e wa0x6e commented Oct 1, 2023

Add a new workflow which will:

  • bump the package.json version (bump version type will depends on the commit messages, follow semver)
  • create a new tag with the new version

Fix #9

@ChaituVR
Copy link
Member

ChaituVR commented Oct 3, 2023

But a new tag will be already created with #13 right?

@wa0x6e
Copy link
Contributor Author

wa0x6e commented Oct 3, 2023

No, #13 only create a github release, this pr bump version and creates the tag

@ChaituVR
Copy link
Member

ChaituVR commented Oct 3, 2023

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

Comment on lines 32 to 38
- 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 }}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

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.

@wa0x6e
Copy link
Contributor Author

wa0x6e commented Oct 3, 2023

Single action is #14


jobs:
bump-version:
name: Bump Version on master
Copy link
Member

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.? 🤔

Copy link
Contributor Author

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

Copy link

@Todmy Todmy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@wa0x6e
Copy link
Contributor Author

wa0x6e commented Nov 6, 2023

These actions do not seem to work well with protected branch. Maybe explore a more all in one solution such as release it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: add new workflow to bump version
3 participants