-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
PEP 621: Migrate from setup.py and setup.cfg to pyproject.toml #1875
Conversation
Ideally, it would be nice to first fix the test suite before continuing modernizing linting/building. What do you think? |
cbaf045
to
31989ef
Compare
ced2e12
to
75ccb2c
Compare
75ccb2c
to
bf2f967
Compare
063f60a
to
5c05572
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks rather good, thanks!
.github/workflows/test.yml
Outdated
@@ -63,7 +63,8 @@ jobs: | |||
python -m pip install --upgrade pip setuptools wheel | |||
pip install coverage requests tox tox-gh-actions | |||
pip install django==${{ matrix.django-version }} elasticsearch==${{ matrix.elastic-version }} | |||
python setup.py clean build install | |||
# python setup.py clean build install |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you intend to keep this commented line?
@@ -20,8 +20,8 @@ jobs: | |||
strategy: | |||
fail-fast: false | |||
matrix: # https://docs.djangoproject.com/en/stable/faq/install/#what-python-version-can-i-use-with-django | |||
django-version: ["3.2", "4.2", "5.0"] | |||
python-version: ["3.8", "3.9"] # , "3.10", "3.11", "3.12"] # Whoosh issues with Py3.10+ | |||
django-version: ["3.2", "4.2"] # , "5.0"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For what reason did you have to remove "5.0" from the matrix?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It currently fails.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh yes, sorry, I forgot it was also excluded on current master.
Migrate
setup.py
tosetup.cfg
using setuptools-py2cfg plus manual modifications. Then migratesetup.cfg
topyproject.toml
using ini2toml to do the file conversion and running pyproject-fmt and then validate-pyproject in pre-commit to validate the results.