-
Notifications
You must be signed in to change notification settings - Fork 90
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
Recommend using twine check
instead
#122
Conversation
README.rst
Outdated
@@ -44,4 +22,7 @@ Code of Conduct | |||
Everyone interacting in the readme_renderer project's codebases, issue trackers, | |||
chat rooms, and mailing lists is expected to follow the `PyPA Code of Conduct`_. | |||
|
|||
|
|||
.. |twine check| replace:: ``twine check`` | |||
.. _twine check: https://github.com/pypa/twine#twine-check |
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.
Hmm, perhaps it would be better to link to the new section that pypa/packaging.python.org#554 is adding instead. I'll update this after that's merged.
7f4e1d7
to
35c0885
Compare
Why exactly is this deprecated? |
Probably because not everyone is using setup.py, so it’s becoming less central to the workflow (somewhat like how tox replaced setup.py test). |
The `check` command was ported to `twine check` in 2018. Refs: pypa#122 Python 3.10 and 3.11 have marked `distutils` as deprecated. Refs: https://peps.python.org/pep-0632/#specification Signed-off-by: Mike Fiedler <[email protected]>
The `check` command was ported to `twine check` in 2018. Refs: #122 Python 3.10 and 3.11 have marked `distutils` as deprecated. Refs: https://peps.python.org/pep-0632/#specification Signed-off-by: Mike Fiedler <[email protected]>
This PR updates the README for this repo to recommend using the new
twine check
command instead ofsetup.py check -r -s
.It also adds a deprecation notice for the old command.
This should be merged once the new version of Twine is released.