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

Test under 3.11-dev #2302

Merged
merged 10 commits into from
Sep 29, 2022
Merged

Test under 3.11-dev #2302

merged 10 commits into from
Sep 29, 2022

Conversation

j178
Copy link
Member

@j178 j178 commented Jul 13, 2022

Added a few warning filters because:

  • _models.py and test_multiplart.py are relying on deprecatedcgi
  • certifi is using deprecated importlib.resources.path
  • rich is using deprecated sre_constants

@Kludex
Copy link
Member

Kludex commented Jul 13, 2022

Should we use 3.11-dev tag instead? So when beta 4 is available on python-setup we get it automatically and can act on it?

@j178 j178 changed the title Test under 3.11.0-beta.3 Test under 3.11-dev Jul 13, 2022
@j178
Copy link
Member Author

j178 commented Jul 13, 2022

Good idea, I don't know we can use 3.11-dev actually, updated.

@florimondmanca
Copy link
Member

florimondmanca commented Jul 14, 2022

Thanks for this! It's up to us to solve the cgi warning so I opened #2309 for this. OTOH the 2 other warnings are in transient dependencies so ignoring them for now seems OK.

@tomchristie
Copy link
Member

I've not looked into why but seems like the coverage check is failing on 3.11 but not other Python versions...

https://github.com/encode/httpx/actions/runs/3150283298/jobs/5122888953

scripts/install Outdated Show resolved Hide resolved
@j178
Copy link
Member Author

j178 commented Sep 29, 2022

@tomchristie This is a bug when coverage is working under 3.11: nedbat/coveragepy#1419. It should be fixed by bumping coverage version.

@tomchristie tomchristie merged commit 8088fc7 into encode:master Sep 29, 2022
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.

4 participants