-
Notifications
You must be signed in to change notification settings - Fork 210
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
chore(ci): upload release artifacts on tag builds #4023
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4023 +/- ##
=======================================
Coverage 57.68% 57.68%
=======================================
Files 193 193
Lines 42907 42907
=======================================
Hits 24750 24750
Misses 18157 18157 ☔ View full report in Codecov by Sentry. |
"upload artifacts" we already have is not for release artifacts.
c81a1a0
to
fbad52d
Compare
I think this will create the GH release by itself? That way we don't have as much control over the release text etc. and might forget to fix it up later. Could we at least add a good automatic change log generator because this will prevent me from using GitHub's builtin one? |
I'm not sure about the exact behavior, but releases can be edited after the fact, even deleted , set as pre-release, etc. so I don't think it's a big deal. |
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 don't have much experience with this myself but looks good as far as I can tell
Git push to origin failed for releases/v0.2 with exitcode 1 |
Fix:
"upload artifacts" we already have is not for uploading release artifacts.