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

Document how to release #765

Closed
wants to merge 1 commit into from
Closed

Conversation

tomwhite
Copy link
Collaborator

Fixes #691

Copy link
Collaborator

@jeromekelleher jeromekelleher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, but I'm not sure I see much value in making maintainers go through these steps manually. Couldn't we add these as an extra github action or something, so that making a release is just "use the UI on GitHub check that all the actions succeed?"


With the latest main branch checked out locally, tag the release::

VERSION=x.y.z
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is assuming you have the main repo set as your origin remote rather than your fork, doesn't it? I'd find this confusing.

Any reason not to use the GitHub UI for tagging a release? It gives you a nice interface for making release notes (also seems to be some new features for automating the generation of release notes), but I haven't looked into it .

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, I see, there's a test pypi bit. Is that necessary? I mean, we spend a lot of time making tests and CI workflows so we don't have to worry about this stuff. Personally I wouldn't bother with checking stuff out locally, I don't see what it gets us on top of all the testing we already do.

@tomwhite
Copy link
Collaborator Author

@jeromekelleher you are right, we should be aiming to do all of the release steps without using a local machine. We already test the artifacts using the scripts I documented here (see https://github.com/pystatgen/sgkit/blob/main/.github/workflows/wheels.yml), so we could make the release via the GitHub UI as you suggest.

The only step that still needs to be done from a local machine is the documentation, but we should add that to the GitHub action too. I've opened #768, which we should do as a part of (or before) the next release.

I'm closing this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document how to release
2 participants