-
Notifications
You must be signed in to change notification settings - Fork 2.1k
🕺 Migrate setup configuration from setup.py
to setup.cfg
and make rich
an optional dep
#3403
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
Conversation
@@ -1,4 +1,3 @@ | |||
include settings.ini |
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.
there is no settings.ini
in the package
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
setup.py
to setup.cfg
setup.py
to setup.cfg
setup.py
to setup.cfg
setup.py
to setup.cfg
and make rich
an optional dep
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.
Nice clean up! LGTM with some nits.
|
||
This ensures that future patch releases (`v{major}.{minor}.1`, `v{major}.{minor}.2`, etc.) can be made separately from `main`. | ||
|
||
#### 9. Create the wheels for your release |
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.
FYI I automate some of these steps with make
here: https://github.com/huggingface/alignment-handbook/blob/main/Makefile#L18
Might be handy if you're sick of manually writing all the commands out
git checkout -b release-v{major}.{minor} | ||
``` | ||
|
||
#### 3. Change the version in the following files |
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.
You could automate some of these steps with a script like this: https://github.com/huggingface/alignment-handbook/blob/main/src/alignment/release.py
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.
Thanks for the pointer! It might be a good idea to copy this into scripts
, I'll do it in a future PR.
[options] | ||
packages = find: | ||
python_requires = >=3.9 |
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.
FYI this is set for EOL in October
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.
Related: python 3.13 should be soon supported: #2593
setup.py
tosetup.cfg
to adopt a declarative, standardized, and more maintainable setup format. This aligns with modern Python packaging practices recommended by PyPA and improves readability, tool compatibility, and static analysis support.setup.py
is now minimal.gives