Skip to content
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

Release CI enhancements after 2.5.0-M1 #1374

Merged
merged 2 commits into from
Sep 20, 2023
Merged

Release CI enhancements after 2.5.0-M1 #1374

merged 2 commits into from
Sep 20, 2023

Conversation

jponge
Copy link
Member

@jponge jponge commented Sep 20, 2023

A few fixes based on observations of what worked, what didn't, and what can be improved.

We're essentially back to a separate workflow to deploy to Maven Central, given how unreliable the gateway can be.
The deployment of 2.5.0-M1 had failed in CI on the Maven side due to a gateway error, but the artifacts actually made it to Maven Central.

Given how unreliable pushing to Maven Central is, we
are better off with a release workflow that makes a source
release, while another (repeatable) workflow can do the
deployment steps. In fact a deployment can fail due to a gateway
error while the artifacts actually end up on Maven Central.
@jponge jponge added the ci/cd/build CI/CD pipeline and build label Sep 20, 2023
@jponge jponge added this to the 2.5.0 milestone Sep 20, 2023
@codecov
Copy link

codecov bot commented Sep 20, 2023

Codecov Report

Merging #1374 (c4467f4) into main (168ac23) will decrease coverage by 0.06%.
Report is 13 commits behind head on main.
The diff coverage is n/a.

❗ Current head c4467f4 differs from pull request most recent head c43fc92. Consider uploading reports for the commit c43fc92 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #1374      +/-   ##
============================================
- Coverage     89.22%   89.16%   -0.06%     
+ Complexity     3361     3358       -3     
============================================
  Files           459      459              
  Lines         13438    13438              
  Branches       1655     1655              
============================================
- Hits          11990    11982       -8     
- Misses          807      811       +4     
- Partials        641      645       +4     

see 13 files with indirect coverage changes

@jponge jponge merged commit 75305d0 into main Sep 20, 2023
5 checks passed
@jponge jponge deleted the ci/fixes-after-2.5.0-M1 branch September 20, 2023 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/cd/build CI/CD pipeline and build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants