-
-
Notifications
You must be signed in to change notification settings - Fork 108
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
Migrate from setuptools + setuptools_scm to hatchling + hatch-vcs #413
Conversation
Codecov Report
@@ Coverage Diff @@
## main #413 +/- ##
=======================================
Coverage 90.07% 90.07%
=======================================
Files 12 12
Lines 2186 2186
=======================================
Hits 1969 1969
Misses 217 217
Flags with carried forward coverage won't be shown. Click here to find out more. Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
I’m curious what you found appealing about hatchling that made it worth using over something more well known like poetry. |
Does renovatebot support hatchling? |
Well first of all, I'd like to move away from setuptools which is getting a bit old (or " Hatchling is standards compliant, and specifically built based on the new packaging standards for build backends (PEP 517/PEP 660) and the format for metadata declaration (PEP 621/PEP 631). Also I haven't used Poetry much, so it doesn't gain much familiarity points. And I've seen a lot of interest and other projects moving over, here's a few: https://hatch.pypa.io/latest/users/ (It's also the default option in this PyPA tutorial https://packaging.python.org/en/latest/tutorials/packaging-projects/.)
No, renovatebot doesn't support PEP 621 / hatchling: renovatebot/renovate#10187 But it doesn't matter as we're not pinning any dependencies there (as libraries shouldn't). We're only using it for GitHub Actions. |
Any update on this? |
@djmattyg007 Do you have any further questions or concerns? |
Nope. I was just curious. |
Good stuff, let's get this merged! 🚀 |
Changes proposed in this pull request: