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

⬆ Upgrade required Python version to >= 3.6.1, needed by typing.Deque, used by Pydantic #2733

Merged
merged 2 commits into from Oct 5, 2021

Conversation

hukkin
Copy link
Contributor

@hukkin hukkin commented Feb 1, 2021

Require Python >=3.6.1 because pydantic requires it. See #2732 and the pydantic PR pydantic/pydantic#2306

Related to: #2732

@codecov
Copy link

codecov bot commented Feb 1, 2021

Codecov Report

Merging #2733 (c3141f0) into master (655db2a) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #2733   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          410       410           
  Lines        10281     10281           
=========================================
  Hits         10281     10281           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 655db2a...c3141f0. Read the comment docs.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2021

📝 Docs preview for commit 42016dd at: https://60174ca20d9f51d716aa27ab--fastapi.netlify.app

@tiangolo tiangolo changed the title Fix python_requires metadata to require >=3.6.1 ⬆ Upgrade required Python version to >= 3.6.1, needed by typing.Deque, used by Pydantic Oct 5, 2021
@tiangolo tiangolo merged commit cf5e675 into tiangolo:master Oct 5, 2021
@hukkin hukkin deleted the patch-1 branch October 5, 2021 08:49
@tiangolo
Copy link
Owner

tiangolo commented Oct 5, 2021

Thanks for your contribution @hukkin! 🍰

This is included in FastAPI 0.68.3, just released. 🎉

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

3 participants