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

[repo] Post-release automation improvements #5664

Merged

Conversation

CodeBlanch
Copy link
Member

Changes

  • When the GitHub Release is published at the end of the release process the automation will now perform two tasks:
    • Call the Core version update workflow on contrib
    • Post a final message on the prepare release PR announcing the packages are available on NuGet

Details

Test runs:

@CodeBlanch CodeBlanch added the infra Infra work - CI/CD, code coverage, linters label May 30, 2024
@CodeBlanch CodeBlanch requested a review from a team May 30, 2024 18:16
Copy link

codecov bot commented May 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.48%. Comparing base (6250307) to head (524a78e).
Report is 247 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #5664      +/-   ##
==========================================
+ Coverage   83.38%   85.48%   +2.10%     
==========================================
  Files         297      254      -43     
  Lines       12531    11054    -1477     
==========================================
- Hits        10449     9450     -999     
+ Misses       2082     1604     -478     
Flag Coverage Δ
unittests ?
unittests-Project-Experimental 85.27% <ø> (?)
unittests-Project-Stable 85.30% <ø> (?)
unittests-Solution 85.47% <ø> (?)
unittests-UnstableCoreLibraries-Experimental 20.13% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 114 files with indirect coverage changes

@vishweshbankwar
Copy link
Member

@CodeBlanch - Should we change this message? CodeBlanchOrg#23 (comment). make it explicit that more steps need to be followed for pushing to nuget?

This seems to look like there is no step in between the following two steps.

image

@CodeBlanch
Copy link
Member Author

@CodeBlanch - Should we change this message? CodeBlanchOrg#23 (comment). make it explicit that more steps need to be followed for pushing to nuget?

This seems to look like there is no step in between the following two steps.

@vishweshbankwar Let me come back to this. I'm going to try to add more into the flow so you can do something like post "/PushPackages" in between those steps to automate that part too. I'll update the messages as part of that.

@CodeBlanch CodeBlanch merged commit 4d02fe1 into open-telemetry:main Jun 3, 2024
50 checks passed
@CodeBlanch CodeBlanch deleted the repo-post-release-contrib-update branch June 3, 2024 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infra Infra work - CI/CD, code coverage, linters
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants