-
Notifications
You must be signed in to change notification settings - Fork 178
Updated release workflow #1060
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
Updated release workflow #1060
Conversation
| @@ -1,37 +0,0 @@ | |||
| name: Release (TestPyPI) | |||
|
|
|||
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.
@mgyucht Is this workflow still relevant/required?
|
If integration tests don't run automatically, an authorized user can run them manually by following the instructions below: Trigger: Inputs:
Checks will be approved automatically on success. |
mgyucht
left a comment
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.
This LGTM.
We should keep the TestPyPI workflow if we can. It is useful in case we need to modify the release workflow; otherwise, there is no way to test changes to it. We struggle for this reason with testing the Java release; the only way to test is by actually making a release, and hopefully it is the version you like. (We can kind of work around this by not automatically "deploying" Java releases during tests, but you are still testing in a production environment.)
What changes are proposed in this pull request?
Updated the Release workflow.
Deleted some unused workflows:
How is this tested?
N/A
NO_CHANGELOG=true