Skip to content

Switch from requirements files to pyproject.toml#2305

Merged
srittau merged 3 commits into
python:mainfrom
srittau:pyproject
Jun 11, 2026
Merged

Switch from requirements files to pyproject.toml#2305
srittau merged 3 commits into
python:mainfrom
srittau:pyproject

Conversation

@srittau

@srittau srittau commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@srittau srittau marked this pull request as ready for review June 11, 2026 10:05
@srittau

srittau commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator Author

This works: https://typing--2305.org.readthedocs.build/en/2305/, also tested the build locally (make -C docs venv && make -C docs html). uv is used only by readthedocs internally as far as I understand, not required locally by developers.

Comment thread pyproject.toml
requires-python = ">= 3.12" # for building the documentation etc.

[dependency-groups]
test = [

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if you call this group dev rather than test, then uv will automatically install it when you're running tests etc. locally using uv run commands (it's treated as a "default dependency group" for local development)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's why I was explicitly not naming it dev, so that it doesn't get installed by default when building docs in this doc-centric repository. But I'll defer to you what the best option is.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, fair enough! I don't feel strongly

@srittau srittau merged commit 14698d2 into python:main Jun 11, 2026
5 checks passed
@srittau srittau deleted the pyproject branch June 11, 2026 11:17
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 this pull request may close these issues.

2 participants