Skip to content

Migrate dependency management to PEP 735 + Add sphinx build CI check#98

Merged
Kalmat merged 1 commit into
Kalmat:masterfrom
Avasam:PEP-735
Jun 16, 2026
Merged

Migrate dependency management to PEP 735 + Add sphinx build CI check#98
Kalmat merged 1 commit into
Kalmat:masterfrom
Avasam:PEP-735

Conversation

@Avasam

@Avasam Avasam commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

https://peps.python.org/pep-0735/

A nice side effect of this is that using uv sync for local development now properly automatically picks-up development dependencies, and you don't leak your dev dependencies as an extra.

Added sphinx build CI check to help prevent breakage. Also done in #100

@Avasam Avasam requested a review from Kalmat June 15, 2026 22:01
@Avasam

Avasam commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator Author

@Kalmat I think you should enable Build pull requests for this project in https://app.readthedocs.org/dashboard/pywinctl/edit/
image

For PRs to validate ReadTheDocs

See: https://docs.readthedocs.com/platform/stable/pull-requests.html

@Avasam Avasam changed the title Migrate dependency management to PEP 735 Migrate dependency management to PEP 735 + Add sphinx build CI check Jun 15, 2026
@Kalmat

Kalmat commented Jun 16, 2026

Copy link
Copy Markdown
Owner

HI! It's a long long time no see you!

I've been out for a long long time too, sorry

I have enabled that option. Not sure what it means, but I fully trust your criteria.

Thanks!

@Kalmat Kalmat merged commit fb20014 into Kalmat:master Jun 16, 2026
5 checks passed
@Avasam Avasam deleted the PEP-735 branch June 16, 2026 18:47
@Avasam

Avasam commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator Author

Been a while indeed, you basically see me anytime I plan an upgrade to AutoSplit that involves PyWinCtl :P (or if you need me for any static typing issue).

I have enabled that option. Not sure what it means, but I fully trust your criteria.

It adds this check !
image

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