Conversation
28c1fd9 to
475ad8c
Compare
There was a problem hiding this comment.
Pull request overview
This PR modernizes the Python packaging configuration by migrating from the legacy setup.py format to the modern pyproject.toml standard (PEP 621). The changes also remove Pipenv in favor of standard pip-based dependency management, update CI/CD from CircleCI to GitHub Actions, and bump the minimum Python version requirement from 3.7 to 3.8. The package version is also incremented from 0.0.4 to 0.0.5.
Changes:
- Migrated packaging configuration from setup.py to pyproject.toml following PEP 621 standards
- Removed Pipenv (Pipfile/Pipfile.lock) and requirements.txt in favor of pyproject.toml's optional dependencies
- Replaced CircleCI configuration with GitHub Actions workflow
- Updated README badges to reflect new CI system and improved black badge formatting
- Bumped minimum Python version from 3.7 to 3.8 and package version from 0.0.4 to 0.0.5
Reviewed changes
Copilot reviewed 9 out of 11 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| setup.py | Removed legacy setuptools configuration file |
| setup.cfg | Removed legacy flake8 configuration file |
| requirements.txt | Removed empty requirements file |
| pyproject.toml | Added modern packaging configuration with build system, project metadata, optional dependencies, and tool configurations |
| aiosocketpool/version.py | Bumped version from 0.0.4 to 0.0.5 |
| README.md | Updated Python version requirement to 3.8+ and replaced CircleCI badge with GitHub Actions CI badge |
| Pipfile.lock | Removed Pipenv lock file |
| Pipfile | Removed Pipenv configuration |
| Makefile | Updated lint commands to pass flake8 configuration via CLI and changed mypy target from '.' to specific directories |
| .github/workflows/ci.yml | Updated dependency installation to use pyproject.toml extras and removed obsolete cache paths |
| .circleci/config.yml | Removed CircleCI configuration in favor of GitHub Actions |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
475ad8c to
49acc1f
Compare
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 11 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 10 out of 12 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Pull request overview
This PR modernizes the Python packaging configuration by migrating from the legacy setup.py format to the modern pyproject.toml standard (PEP 621). The changes also remove Pipenv in favor of standard pip-based dependency management, update CI/CD from CircleCI to GitHub Actions, and bump the minimum Python version requirement from 3.7 to 3.8. The package version is also incremented from 0.0.4 to 0.0.5.
Changes