-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Add a GitHub workflow for finalizing a release #6998
Conversation
This automates the second half of the release process (the first being automated by `prepare-release.yml`). After this workflow completes, the only action that should remain for the releaser to do is to merge the `dev-release-*` pull request. We can't do that as part of this workflow, because CI has to finish first, and it seems pointless to create another workflow just to merge 1 PR. Apply some of the aspects of this pipeline to `prepare-release.yml` as well: * Make the release notes extraction process more sophisticated to work around GitHub's frustrating handling of line breaks in PR and release descriptions. * Use GitHub app credentials in order to be able to trigger other pipelines.
eae68da
to
55f23cc
Compare
Codecov Report
@@ Coverage Diff @@
## develop #6998 +/- ##
===========================================
- Coverage 82.60% 82.54% -0.06%
===========================================
Files 360 360
Lines 38922 38977 +55
Branches 3570 3577 +7
===========================================
+ Hits 32150 32174 +24
- Misses 6772 6803 +31
|
I had an issue yesterday with using this workflow: https://github.com/opencv/cvat/actions/runs/6618431921/job/17976838325 |
Looks like it ran afoul of the branch protection rules. I've now added cvat-bot to the list of people who are allowed to push to |
This automates the second half of the release process (the first being automated by `prepare-release.yml`). After this workflow completes, the only action that should remain for the releaser to do is to merge the `dev-release-*` pull request. We can't do that as part of this workflow, because CI has to finish first, and it seems pointless to create another workflow just to merge 1 PR. Apply some of the aspects of this pipeline to `prepare-release.yml` as well: * Make the release notes extraction process more sophisticated to work around GitHub's frustrating handling of line breaks in PR and release descriptions. * Use GitHub app credentials in order to be able to trigger other pipelines.
Motivation and context
This automates the second half of the release process (the first being automated by
prepare-release.yml
). After this workflow completes, the only action that should remain for the releaser to do is to merge thedev-release-*
pull request. We can't do that as part of this workflow, because CI has to finish first, and it seems pointless to create another workflow just to merge 1 PR.Apply some of the aspects of this pipeline to
prepare-release.yml
as well:Make the release notes extraction process more sophisticated to work around GitHub's frustrating handling of line breaks in PR and release descriptions.
Use GitHub app credentials in order to be able to trigger other pipelines.
How has this been tested?
I tried the pipelines on my CVAT fork.
Checklist
develop
branch[ ] I have created a changelog fragment[ ] I have updated the documentation accordingly[ ] I have added tests to cover my changes[ ] I have linked related issues (see GitHub docs)[ ] I have increased versions of npm packages if it is necessary(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)
License
Feel free to contact the maintainers if that's a concern.