-
Notifications
You must be signed in to change notification settings - Fork 49
chore: improvements to release workflow #1451
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
Conversation
Signed-off-by: Todd Baert <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1451 +/- ##
============================================
+ Coverage 92.88% 93.31% +0.42%
- Complexity 483 485 +2
============================================
Files 45 45
Lines 1167 1167
Branches 101 101
============================================
+ Hits 1084 1089 +5
+ Misses 53 48 -5
Partials 30 30
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Unfortunately, I don't think this change will help. You'll be able to rerun the workflow but it won't attempt to republish. |
permissions: | ||
contents: read | ||
needs: release-please | ||
if: ${{ fromJSON(needs.release-please.outputs.release_created || false) }} |
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.
You shouldn't need the fromJSON
workaround here since you're already handling it on line 28. I suppose it doesn't hurt to be extra careful though.
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.
Ya, we've keep it like this in most places.
That doesn't seem to be what the doc says:
|
|
A few improvements to our release workflow: