暗黙的namespace package(PEP 420)に変更し、Python 3.13-3.14/Django 5.2をサポート#18
暗黙的namespace package(PEP 420)に変更し、Python 3.13-3.14/Django 5.2をサポート#18kashewnuts wants to merge 3 commits intomasterfrom
Conversation
beproud/__init__.pyとbeproud/django/__init__.pyを削除して暗黙的namespace packageに変更。bpcommonsとのファイル競合を構造的に解消。 pyproject.tomlのincludeをbeproud.django.notify*に限定し、バージョンを 0.51に更新。Python 3.9を削除し、Python 3.13-3.14、Django 5.2、 Celery 5.6を追加。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
bpcommons との競合を避けるために beproud.* を暗黙的 namespace package (PEP 420) に移行しつつ、サポートする Python/Django/Celery のバージョン範囲を更新するPRです。
Changes:
beproud/__init__.pyとbeproud/django/__init__.pyを削除し、PEP 420 の暗黙的 namespace package に移行pyproject.tomlのパッケージ include をbeproud.django.notify*に限定し、0.51 にバージョン更新・対応バージョン表記更新- tox のテストマトリクスを Python 3.10–3.14 / Django 4.2, 5.2 / Celery 5.3, 5.6 に拡張
Reviewed changes
Copilot reviewed 3 out of 5 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| tox.ini | tox/envlist と tox-gh-actions の Python/Django マッピングを更新 |
| pyproject.toml | requires-python / classifiers 更新、パッケージ探索 include を限定、version を 0.51 に更新 |
| CHANGES.rst | 0.51 の変更点を追記 |
| beproud/init.py | 暗黙的 namespace package 化のため削除 |
| beproud/django/init.py | 暗黙的 namespace package 化のため削除 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
GitHub Actionsのmatrixを更新してPython 3.13-3.14とDjango 5.2を追加し、 Python 3.13-3.14とDjango 4.2の組み合わせを除外。actions/checkout@v6と actions/setup-python@v6に更新。pyproject.tomlからDjango 5.1を削除。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
| matrix: | ||
| python-version: ['3.9', '3.10', '3.11', '3.12'] | ||
| django-version: ['4.2'] | ||
| python-version: ['3.10', '3.11', '3.12', '3.13', '3.14'] |
There was a problem hiding this comment.
MEMO: タイトルの通りPythonとDjangoの対象バージョンを増やしたので更新
| @@ -1,28 +1,30 @@ | |||
| [build-system] | |||
| requires = ["setuptools>=61.0.0", "wheel"] | |||
| requires = ["setuptools>=77.0.3", "wheel"] | |||
There was a problem hiding this comment.
MEMO: バージョン情報を更新
ref: https://packaging.python.org/ja/latest/guides/writing-pyproject-toml/
|
|
||
| [tool.setuptools.packages.find] | ||
| where = ["."] | ||
| include = ["beproud.django.notify*"] |
There was a problem hiding this comment.
MEMO: このPRではこの行でincludeの追加と、 __init__.py を削除したのがキモ
ref: https://packaging.python.org/ja/latest/guides/packaging-namespace-packages/#native-namespace-packages
| pytest | ||
| pytest-django | ||
| pytest-pythonpath | ||
| setuptools |
There was a problem hiding this comment.
MEMO: ソースコード上利用している箇所もなく、toxも通っているので削除しました。
| - uses: actions/checkout@v6 | ||
|
|
||
| # ジョブのPython環境を設定 | ||
| - name: Set up Python ${{ matrix.python-version }} | ||
| uses: actions/setup-python@v5 | ||
| uses: actions/setup-python@v6 |
There was a problem hiding this comment.
MEMO: actionsが古くなっていたので更新
kashewnuts
force-pushed
the
implicit-namespace
branch
from
02486e4 to
f15d52b
Compare
1 participant
何を達成したいのか、なぜその変更をしたか
具体的な変更内容
実装時に確認したこと
python -m build .とtwine check --strict dist/*が実行できること特にレビューしてほしいこと