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

Skip to content

Log email-validator not installed only when used#946

Merged
tiangolo merged 2 commits into
masterfrom
email-validator-warn-only-when-used
Feb 4, 2020
Merged

Log email-validator not installed only when used#946
tiangolo merged 2 commits into
masterfrom
email-validator-warn-only-when-used

Conversation

@tiangolo

@tiangolo tiangolo commented Feb 4, 2020

Copy link
Copy Markdown
Member

🔇 Log email-validator not installed only when used

@codecov

codecov Bot commented Feb 4, 2020

Copy link
Copy Markdown

Codecov Report

Merging #946 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #946   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files         293    293           
  Lines        7692   7785   +93     
=====================================
+ Hits         7692   7785   +93
Impacted Files Coverage Δ
fastapi/openapi/models.py 100% <100%> (ø) ⬆️
...est_additional_responses_custom_validationerror.py 100% <0%> (ø) ⬆️
docs/src/app_testing/tutorial003.py 100% <0%> (ø) ⬆️
...t_query_params_str_validations/test_tutorial001.py 100% <0%> (ø) ⬆️
tests/test_custom_swagger_ui_redirect.py 100% <0%> (ø) ⬆️
docs/src/first_steps/tutorial001.py 100% <0%> (ø) ⬆️
tests/test_additional_responses_router.py 100% <0%> (ø) ⬆️
docs/src/application_configuration/tutorial001.py 100% <0%> (ø) ⬆️
...test_tutorial/test_body_fields/test_tutorial001.py 100% <0%> (ø) ⬆️
tests/test_swagger_ui_init_oauth.py 100% <0%> (ø) ⬆️
... and 79 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 55afb70...1155006. Read the comment docs.

@tiangolo tiangolo merged commit 2e0a102 into master Feb 4, 2020
@tiangolo tiangolo deleted the email-validator-warn-only-when-used branch February 4, 2020 04:31
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.

1 participant