-
Notifications
You must be signed in to change notification settings - Fork 62
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
Implement PEP-518 #295
Comments
Yes, agreed. Only thing to watch out for is whether we need an extra dependency to support the .toml syntax. I assume that's OK. |
usually projects use a |
any progress? |
PEP 621 support would be even better! |
I don't have an immediate need for this, so I am not likely to work on this in the near future. But if someone wants to make a PR, that would be fine. Question is which toml parser to use. @graingert mentions I wonder: what would we do with it? Only use this to read the Or do we also for example read and write the version number in When we read config options, I would say the following is a good order, where the first file where an option is found, wins:
|
|
toml doesn't support toml v1 syntax. pip is also moving to tomli. pypa/pip#10035 |
In the long run I suppose it needs to support updating the version no matter if it is in setup.cfg or pyproject.toml. But looking into pyproject.toml at the moment, it seems like setuptools doesn't fully support it, so you still need a setup.cfg. I think the only build project that supports not having a setup.cfg is Poetry? |
And flit
…On Tue, 10 Aug 2021, 12:54 Lennart Regebro, ***@***.***> wrote:
In the long run I suppose it needs to support updating the version no
matter if it is in setup.cfg or pyproject.toml.
But looking into pyproject.toml at the moment, it seems like setuptools
doesn't fully support it. I think the only build project that supports not
having a setup.cfg is Poetry?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#295 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADFATFLDL7XVFRQ7MPF733T4EHODANCNFSM4GCIGAUA>
.
|
Projects have now for the most part switched to tomli |
Here's a related issue about pep621 #373 |
A PR in that direction would be welcome. Does anyone want to take this on? Other approach would be to drop Python 2.7 support. |
Since Python 2 support ended over one and a half years ago, I fully endorese dropping Python 2 support. |
I tried making a py2/py3 toml compat lib here https://github.com/graingert/read-toml/blob/default/read_toml/_impl.py#L23 but it's tricky getting it to type check |
I think the way to go about dropping py2 is to cut a release with whatever you've got on the default branch then apply pyupgrade and python_requires with https://pre-commit.ci |
I have created issue #374 for future plans. |
Version 8.0.0a1 supports writing the version to But for releasing you still need a |
Fixed in #415 |
Having the ability to use pyproject.toml instead of setup.cfg would be a "nice to have" feature.
The text was updated successfully, but these errors were encountered: