-
Notifications
You must be signed in to change notification settings - Fork 5
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
Drop support for legacy python #8
Conversation
50d8e0d
to
bcff387
Compare
This appears to be needed for compatibility with newer Python versions. It's unclear why this wasn't previously showing up in CI as fresh runs without code changes do now hit issues which this change appears to resolve. As we're updating `black` here this commit also includes a layout change to placate the formatter.
bcff387
to
f8e008c
Compare
f8e008c
to
4170560
Compare
@@ -12,7 +12,7 @@ jobs: | |||
runs-on: ubuntu-20.04 | |||
strategy: | |||
matrix: | |||
python-version: [3.6, 3.7, 3.8, 3.9] | |||
python-version: [3.7, 3.8, 3.9] |
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 want to add 3.10 to this?
@@ -1,10 +1,9 @@ | |||
[tox] | |||
isolated_build = true | |||
envlist = py{36,37,38,39}-django{22,30,31,32} | |||
envlist = py{37,38,39}-django{22,30,31,32} |
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.
Can we also try Django 4 here as well? If it doesn't work without further changes don't worry about it, but I think it wasn't a huge change so it might Just Work.
Well that'll teach me to review in chronological order. Merging now. |
Fixes #6
This ends up updating both the linting tools and
poetry
used in CI to fix some compile issues that CI was seeing. I'm not sure why those issues were only appearing, but I confirmed via #10 that they weren't caused by the dependency change.They're bundled here as they end up depending on the dependency change in order for
poetry
to successfully update the linting tools.