-
Notifications
You must be signed in to change notification settings - Fork 370
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
build(release): configure semantic release #289
Conversation
feat(stale-and-close): add new options to change the days before clos…
Remove the stale label when labeled with an exempt one (actions#268)
@hross when you have the time if you can take a look 👍🏻. |
It's going to take me some time to get to this one, especially since it changes the release structure. I will take a look later this week and see if it makes sense (seems cool). |
I haven't gotten to this one yet. Will look next week. |
I still like this change but making release modifications requires me to spend a fair bit of time testing and watching and not breaking things so I have not yet merged this. |
I'm not sure if using angular commit message conventions is the way to go in order to manage the release process. Creating an alpha branch and auto-releasing would be pretty cool but keeping the release process manual means less commit maintenance and lock into the process, plus it's just a tag bump and more of a decision about a major release. Need to think more on this PR. |
e3213d7
to
14d6e9d
Compare
@hross I am not sure to follow up with the issue. |
I don't think we should do this one for now. I don't want to commit this project to a different release strategy from our other actions right now. Maybe we can come back to it at some point, but for now I'm closing to avoid clutter. |
Some infos:
- Get the current tag
- Create the release
- Install the dependencies (use some cache as well)
- Create the release (see above workflow)
- Create a new PR from master to main if there is a new tag (so that you can easily merge the changelog and packages changes)
What needs to be done before merging (by admin):
Closes #288