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 version pin accepted for Flake8, for internal code, to flake8 >=3.8.3,<6.0.0 #4097

Merged
merged 4 commits into from
Aug 26, 2022

Conversation

jamescurtin
Copy link
Contributor

@jamescurtin jamescurtin commented Oct 27, 2021

Flake8 released a new major version, and the pinned upper bound in pyproject.toml needs to be increased to reflect this.

@codecov
Copy link

codecov bot commented Oct 27, 2021

Codecov Report

Merging #4097 (5dd61f4) into master (92181ef) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master     #4097   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          534       535    +1     
  Lines        13811     13830   +19     
=========================================
+ Hits         13811     13830   +19     
Impacted Files Coverage Δ
fastapi/utils.py 100.00% <0.00%> (ø)
tests/test_openapi_query_parameter_extension.py 100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@github-actions
Copy link
Contributor

📝 Docs preview for commit 1976460 at: https://61795884c3e3f015f562d915--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commit 5dd61f4 at: https://630894a72bde1b6d0e2e39b5--fastapi.netlify.app

@tiangolo tiangolo changed the title Increase version range of flake8 to support 4.x releases ⬆ Upgrade version pin accepted for Flake8, for internal code, to flake8 >=3.8.3,<6.0.0 Aug 26, 2022
@tiangolo
Copy link
Member

Thanks @jamescurtin! 🚀

I updated it a bit, now that there's a new version, etc. Thanks for your contribution! 🚀 🍰

@tiangolo tiangolo merged commit d5c8459 into fastapi:master Aug 26, 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.

2 participants