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

[documentation] project.dependencies is not a TOML table #12334

Closed
andrei-korshikov opened this issue Mar 20, 2025 · 3 comments · Fixed by #12388
Closed

[documentation] project.dependencies is not a TOML table #12334

andrei-korshikov opened this issue Mar 20, 2025 · 3 comments · Fixed by #12388
Labels
documentation Improvements or additions to documentation good first issue Good for newcomers

Comments

@andrei-korshikov
Copy link

Project dependencies reads:

The project.dependencies table represents the dependencies that are used when uploading to PyPI or building a wheel.

PEP 621 reads:

Format: Array of PEP 508 strings (dependencies) and a table with values of arrays of PEP 508 strings (optional-dependencies)

So, to my understanding, project.dependencies is an array (kinda list), not a table (map, dictionary, key-value pairs).

@andrei-korshikov
Copy link
Author

Hmm, then Dependency tables are not "tables" but "array and tables"…

@andrei-korshikov
Copy link
Author

Dependency tables reads:

The tool.uv.sources table extends the standard dependency tables

and has dependencies = ["foo"] array sample.

Am I missing something obvious?

@zanieb
Copy link
Member

zanieb commented Mar 20, 2025

We could probably just switch to "field" or something here. I don't think "table" was intended to convey semantic meaning as used.

@zanieb zanieb added documentation Improvements or additions to documentation good first issue Good for newcomers labels Mar 20, 2025
@zanieb zanieb closed this as completed in cdd6de5 Mar 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants