Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add typing.Optional to variables that accept None as value #1731

Merged
tiangolo merged 10 commits into
fastapi:masterfrom
blkst8:add-optional
Aug 3, 2020
Merged

Add typing.Optional to variables that accept None as value #1731
tiangolo merged 10 commits into
fastapi:masterfrom
blkst8:add-optional

Conversation

@blkst8

@blkst8 blkst8 commented Jul 18, 2020

Copy link
Copy Markdown
Contributor

No description provided.

@codecov

codecov Bot commented Jul 18, 2020

Copy link
Copy Markdown

Codecov Report

Merging #1731 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #1731   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          235       235           
  Lines         6989      6987    -2     
=========================================
- Hits          6989      6987    -2     
Impacted Files Coverage Δ
fastapi/openapi/utils.py 100.00% <ø> (ø)
fastapi/security/open_id_connect_url.py 100.00% <ø> (ø)
fastapi/applications.py 100.00% <100.00%> (ø)
fastapi/dependencies/models.py 100.00% <100.00%> (ø)
fastapi/dependencies/utils.py 100.00% <100.00%> (ø)
fastapi/encoders.py 100.00% <100.00%> (ø)
fastapi/exceptions.py 100.00% <100.00%> (ø)
fastapi/param_functions.py 100.00% <100.00%> (ø)
fastapi/params.py 100.00% <100.00%> (ø)
fastapi/routing.py 100.00% <100.00%> (ø)
... and 9 more

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 14e7f7c...0d9b38d. Read the comment docs.

@tiangolo tiangolo merged commit 3223de5 into fastapi:master Aug 3, 2020
@tiangolo

tiangolo commented Aug 3, 2020

Copy link
Copy Markdown
Member

Awesome, 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.

2 participants