Skip to content

Allow hatch-vcs 0.5 (fix #3612) #3613

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

Merged
merged 3 commits into from
May 30, 2025
Merged

Conversation

musicinmybrain
Copy link
Contributor

@musicinmybrain musicinmybrain commented May 29, 2025

Keep SemVer-bounding hatch-vcs as agreed in #3412 (comment), but stop pinning it all the way to the patch release. Fixes #3612.

I’m happy to simply expand this version bound for now, although I would like to suggest that it might not really be necessary to upper-bound the version of hatch-vcs after all. There are 79 packages in Fedora that have hatch-vcs as a build dependency, and python-urllib3 is the only one that upper-bounds the version of hatch-vcs. Furthermore, we found no regressions in any of those packages when we updated from 0.4.0 to 0.5.0.

Keep SemVer-bounding `hatch-vcs` as agreed in
urllib3#3412 (comment),
but stop pinning it all the way to the patch release.
Copy link
Member

@pquentin pquentin left a comment

Choose a reason for hiding this comment

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

Thank you! LGTM. The CI failure is unrelated.

Stories like https://lwn.net/Articles/1020576/ make me reticent to unpin completely, especially since hatch-vcs did not reach 1.0 yet. I've added a commit to pin setuptools-scm, for consistency, as it's way more active than hatch-vcs itself.

@pquentin pquentin merged commit 47ac841 into urllib3:main May 30, 2025
33 of 35 checks passed
@musicinmybrain
Copy link
Contributor Author

musicinmybrain commented May 30, 2025

Stories like https://lwn.net/Articles/1020576/ make me reticent to unpin completely, especially since hatch-vcs did not reach 1.0 yet. I've added a commit to pin setuptools-scm, for consistency, as it's way more active than hatch-vcs itself.

Fair enough. Downstream in Fedora, we will (necessarily) keep removing or editing the pins as needed, and – as in this PR – letting you know whether or not anything catches fire when we do so. 😄

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.

Support hatch-vcs 0.5.0
2 participants