-
Notifications
You must be signed in to change notification settings - Fork 20
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
Change release flow #111
Change release flow #111
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #111 +/- ##
==========================================
- Coverage 59.61% 59.49% -0.13%
==========================================
Files 33 33
Lines 16071 16071
==========================================
- Hits 9581 9561 -20
- Misses 6490 6510 +20 ☔ View full report in Codecov by Sentry. |
description: "Release version, eg:latest, 0.2.1" | ||
required: true | ||
default: "0.0.0" | ||
workflow_dispatch: # select tag when creating |
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.
workflow_dispatch: # select tag when creating | |
on: | |
release: | |
types: [published] |
use this instead?
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.
What would be the intended use case? We have an action further down in the script ( softprops/action-gh-release@v2
) to create a release, and that becomes quite useless if we already have created a release.
And adding on:release
is a bit superfluous. If you create a new release, and the tag does not exist since before a new tag will be created and that will anyway trigger this workflow.
I am actually considering if we should remove workflow_dispatch
- I do not see that many use-cases for it, except possibly if you have yanked a release and want to rebuild a new release for the same tag.
Shall not be needed
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 useful. I think the risk is sort of low, it creates draft releases, that can likely be deleted via GH WebUI if really needed.
This is a proposal to change the release handling somewhat
Today
1.2.3
)If you trigger on branch you will get a faulty tag proposal in draft release that you must correct, tag will be created when release is published
After this PR
If you run the workflow on a branch you will get a release named after the branch, which you likely do not want