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

add support for Django 5.0 and Python 3.12 #1331

Merged
merged 2 commits into from Nov 26, 2023

Conversation

sarahboyce
Copy link
Contributor

@sarahboyce sarahboyce commented Oct 29, 2023

Hello 👋 firstly thank you so much for this package!

As Django 5.0 beta is released (and 5.0 will be released in December), this is hopefully some of the prep work for testing against 5.0. Understand though if you want to wait until the final version is released.

@jschneier
Copy link
Owner

Thanks for the PR. I'm looking forward to dropping everything before 4.2 in accordance with the guidelines put forth by upstream once it is released.

@sarahboyce
Copy link
Contributor Author

Looks like some dependencies are not depreciation free for Python 3.12 (such as dateutil dateutil/dateutil#1314)
Should I remove 3.12 from the test matrix? 🤔

@jschneier
Copy link
Owner

Looks like some dependencies are not depreciation free for Python 3.12 (such as dateutil dateutil/dateutil#1314) Should I remove 3.12 from the test matrix? 🤔

I'm not overly bothered by the deprecation warnings, it seems like the actual failure comes from not installing setuptools automatically into venv which seems to me like a dropbox library issue?

@sarahboyce
Copy link
Contributor Author

I'm not overly bothered by the deprecation warnings, it seems like the actual failure comes from not installing setuptools automatically into venv which seems to me like a dropbox library issue?

Yeah you're right. I created an issue in the dropbox library here: dropbox/dropbox-sdk-python#483
Looks like it is maintained by a dropbox team so 🤞 hopefully they will take a look.

tox.ini Show resolved Hide resolved
@sarahboyce sarahboyce changed the title add support for Django 5.0 add support for Django 5.0 and Python 3.12 Oct 31, 2023
@sarahboyce sarahboyce force-pushed the django-50 branch 2 times, most recently from 96fea47 to 27db534 Compare October 31, 2023 15:09
Copy link
Contributor

@adamchainz adamchainz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for doing this Sarah. @jschneier / @foarsitter could you approve the workflow to run?

.github/workflows/ci.yml Outdated Show resolved Hide resolved
@sarahboyce
Copy link
Contributor Author

I have updated that dropbox tests should be skipped when testing Python 3.12 (ref dropbox/dropbox-sdk-python#483) as not currently supported 👍

@jschneier
Copy link
Owner

For Dropbox does it make sense to install setuptools into the test environment?

@sarahboyce
Copy link
Contributor Author

For Dropbox does it make sense to install setuptools into the test environment?

I think (if I understand correctly) setuptools is already installed in the ci tests.

pip install --upgrade setuptools

For me, the ideal solution is dropbox accepts this PR: dropbox/dropbox-sdk-python#484 and publish a new release on PyPi 🤞
I have faith they will but it might take some time

@jschneier
Copy link
Owner

The skipping Dropbox fix doesn't seem to work :|

@sarahboyce
Copy link
Contributor Author

Sorry @jschneier I should have been testing against my fork from the start 🤦‍♀️
https://github.com/sarahboyce/django-storages/actions/runs/6978703418/job/18990596632?pr=1
I think this is working now 🤞

@jschneier
Copy link
Owner

Sorry @jschneier I should have been testing against my fork from the start 🤦‍♀️ https://github.com/sarahboyce/django-storages/actions/runs/6978703418/job/18990596632?pr=1 I think this is working now 🤞

No worries once this goes in the CI will not be blocked. Lint issue popped up from the fix but that should be the last bit.

Co-authored-by: RachellCalhoun <rachell.calhoun@gmail.com>
Co-authored-by: AmiZya <amizya@gmail.com>
@jschneier jschneier merged commit 70fa8d9 into jschneier:master Nov 26, 2023
19 checks passed
@jschneier
Copy link
Owner

Thanks!

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.

None yet

4 participants