Skip to content
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

Using a top-level table for pytest in pyproject.toml offends PEP 518 #4

Closed
abravalheri opened this issue Dec 23, 2021 · 0 comments · Fixed by #5
Closed

Using a top-level table for pytest in pyproject.toml offends PEP 518 #4

abravalheri opened this issue Dec 23, 2021 · 0 comments · Fixed by #5

Comments

@abravalheri
Copy link
Contributor

abravalheri commented Dec 23, 2021

Hi @jaraco, I am under the impression that using a top-level pytest table is not allowed under PEP 518.

with open(filename) as strm:
defn = toml.load(strm)
return defn["pytest"]["enabler"]

The PEP specifies the tool namespace, so I suppose the best would be reading from tool.pytest.enabler (or tool.pytest-enabler)?

I noticed this issue when I was working on the PEP 621 support for setuptools (and trying to convert setuptools' own setup.cfg file to pyproject.toml).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant