-
Notifications
You must be signed in to change notification settings - Fork 230
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
Add "Bug Tracker" and "Documentation" links to pyproject.toml #826
Add "Bug Tracker" and "Documentation" links to pyproject.toml #826
Conversation
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.
It'd be better to put this metadata in pyproject.toml
.
I think it'll work fine to put this there without moving anything else.
7ec5502
to
689316d
Compare
Hmm it seems like starting a
|
Interesting, looks like we have to migrate a few other properties over
first. If you're up for it, that'd be a great contribution, if not I'll
make time to work on this shortly.
…On Wed, Jun 26, 2024 at 8:38 AM Attila Szakacs ***@***.***> wrote:
Hmm it seems like starting a project block in pyproject.toml conflicts
with setup.py.
py312: packaging backend failed (code=1), with ValueError: invalid pyproject.toml config: `project`.
configuration error: `project` must contain ['name'] properties
—
Reply to this email directly, view it on GitHub
<#826 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAAGBAFDZ2BVGBQETQU3YTZJKY4NAVCNFSM6AAAAABJ5PXZV6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJRGU4DSMJWGU>
.
You are receiving this because you commented.Message ID:
***@***.***>
--
All that is necessary for evil to succeed is for good people to do nothing.
|
Yes, I'd like to try that, but feel free to implement it yourself anytime if you need this sooner. :) |
I won't get to it this weekend, so if you're up for it, please have a shot!
…On Wed, Jun 26, 2024 at 6:03 PM Attila Szakacs ***@***.***> wrote:
Yes, I'd like to try that, but feel free to implement it yourself anytime if you need this sooner. :)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: ***@***.***>
--
All that is necessary for evil to succeed is for good people to do nothing.
|
We've now got a basic pyproject.toml, so you should be able to rebase on main and do this now. |
689316d
to
567a6e2
Compare
They show up on the pypi page on the left, making them easier to find. Based on: https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#urls Signed-off-by: Attila Szakacs <attila.szakacs@axoflow.com>
Thanks, kind of you to leave the urls section to me :) Rebased and updated the PR. |
567a6e2
to
3a32902
Compare
They show up on the pypi page on the left, making them easier to find.
Based on:
https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#urls