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

Replace setup.{cfg,py} with pyproject.toml #216

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

dbohdan
Copy link
Contributor

@dbohdan dbohdan commented Apr 9, 2025

This PR migrates the project from setup.{cfg,py} to pyproject.toml. It removes Trove classifiers for Python versions as redundant, replaces deprecated from aiohttp import wsgi with aiohttp-wsgi, and updates GitHub Actions CI. In GitHub Actions, it adds Python 3.13 tests and separates flake8/docs from tests proper. This gives better indication what kind of error has occurred.

The PR keeps tox.ini because of two tools:

  1. tox 4.13 in Debian 12. tox only added support for native TOML configuration in version 4.21.
  2. Flake8, which doesn't support TOML configuration without a third-party package.

Resolves #215.

@dbohdan dbohdan force-pushed the pyproject branch 5 times, most recently from 4433c44 to 2d2e923 Compare April 9, 2025 19:58
@dbohdan
Copy link
Contributor Author

dbohdan commented Apr 9, 2025

I had to make several corrections and to rebase my branch on your master. The PR is ready now.

@dbohdan dbohdan force-pushed the pyproject branch 3 times, most recently from d735505 to 2460500 Compare April 11, 2025 12:02
This gives better indication what kind of error has occurred.
dbohdan added 3 commits April 11, 2025 12:22
Remove Trove classifiers for specific Python 3 versions as redundant.

Replace `from aiohttp import wsgi` with aiohttp-wsgi.
aiohttp.wsgi was dropped in
aio-libs/aiohttp#1108.

Test in Python 3.13.

This migration keeps tox.ini for two reasons:
1. tox only added support for native TOML configuration
   in recent version 4.21.
2. Flake8 doesn't support TOML configuration
   without a third-party package.
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.

Convert to pyproject
2 participants