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 Process]: Please implement a tag and release workflow and have it fail early if the version is wrong #5539

Closed
Kangie opened this issue Dec 6, 2024 · 5 comments · Fixed by #5536
Assignees

Comments

@Kangie
Copy link

Kangie commented Dec 6, 2024

The version number did not get an update before being tagged as 2.2.0:

version = "2.1.0"

Since #5531 means that the PyPi tarball is unsuitable for me as a dowstream I've had to resort to seding pyproject.toml (etc).

Proposed solution

Please come up with a solution that produces usable GitHub release artefacts.

@snejus
Copy link
Member

snejus commented Dec 6, 2024

This is weird, didn't realize I thought I addressed this issue with 2.1.0 release, but it seems like it's occurring again. Will get this fixed asap.

@snejus snejus self-assigned this Dec 6, 2024
@snejus
Copy link
Member

snejus commented Dec 6, 2024

Could you confirm that moving the tag to the correct commit would fix your issue?

@Kangie
Copy link
Author

Kangie commented Dec 7, 2024

Could you confirm that moving the tag to the correct commit would fix your issue?

I've worked around it downstream using sed - It's more making sure that a future release can't go ahead if the version and tag aren't in alignment to catch this in future :)

@snejus
Copy link
Member

snejus commented Dec 8, 2024

This makes sense. If you have time, feel free to have a look at the PR that fixes this issue.

@Serene-Arc
Copy link
Contributor

This is a weird bug that comes from having the tagging action in the wrong part, which @snejus found and fixed! I introduced it when I made the workflow. It's been an issue for the last few releases and we usually fix it manually. Hopefully this means that it won't be an issue in the future.

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