-
Notifications
You must be signed in to change notification settings - Fork 931
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
Update release workflow #1523
base: main
Are you sure you want to change the base?
Update release workflow #1523
Conversation
4a3113a
to
c985a57
Compare
Signed-off-by: Jeffrey Kinard <jeff@thekinards.com>
@damccorm Could you PTAL when you have a chance? These changes should help alleviate pain points in the release and when examining test failures on scheduled Java PR's (and the release). |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1523 +/- ##
==========================================
Coverage 41.26% 41.27%
+ Complexity 3242 2938 -304
==========================================
Files 771 771
Lines 45127 45127
Branches 4819 4819
==========================================
+ Hits 18622 18624 +2
+ Misses 24938 24936 -2
Partials 1567 1567
|
Yes - though moving forward, could you try to request review only once the PR is ready/checks are passing? It is pretty hard to follow notifications otherwise. Same applies to #1697 for example - I got the review requested notification, but checks were initially failing then there was a series of commits and I'm not sure if you want a review. |
env: | ||
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
TAG_NAME: ${{ steps.variables.outputs.tagName }} | ||
- name: Get releaser identity |
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 like there was an indentation change, which I think may break this workflow - could we revert? It will also make this section much easier to review
Update release workflow to use modular jobs similar to Java PR workflow. Also includes site report for easier failure tracking in release and Java PR workflows.