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

Skip to content

enable branch coverage#3560

Closed
graingert wants to merge 1 commit into
fastapi:masterfrom
graingert:enable-branch-coverage
Closed

enable branch coverage#3560
graingert wants to merge 1 commit into
fastapi:masterfrom
graingert:enable-branch-coverage

Conversation

@graingert

Copy link
Copy Markdown
Contributor

No description provided.

@graingert graingert force-pushed the enable-branch-coverage branch from 9f4086a to 7b1826c Compare July 20, 2021 09:09
@codecov

codecov Bot commented Jul 20, 2021

Copy link
Copy Markdown

Codecov Report

Merging #3560 (7b1826c) into master (0a38c51) will decrease coverage by 0.21%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##            master    #3560      +/-   ##
===========================================
- Coverage   100.00%   99.78%   -0.22%     
===========================================
  Files          254      256       +2     
  Lines         7855     7958     +103     
  Branches         0      344     +344     
===========================================
+ Hits          7855     7941      +86     
- Partials         0       17      +17     
Impacted Files Coverage Δ
fastapi/encoders.py 95.65% <0.00%> (-4.35%) ⬇️
fastapi/openapi/utils.py 97.69% <0.00%> (-2.31%) ⬇️
fastapi/security/oauth2.py 98.41% <0.00%> (-1.59%) ⬇️
fastapi/dependencies/utils.py 98.75% <0.00%> (-1.25%) ⬇️
fastapi/applications.py 99.20% <0.00%> (-0.80%) ⬇️
fastapi/routing.py 99.31% <0.00%> (-0.69%) ⬇️
fastapi/utils.py 100.00% <0.00%> (ø)
tests/utils.py 100.00% <0.00%> (ø)
fastapi/__init__.py 100.00% <0.00%> (ø)

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 0a38c51...7b1826c. Read the comment docs.

@github-actions

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 7b1826c at: https://60f6936ea187b6a09e90ce0e--fastapi.netlify.app

@Kludex

Kludex commented Jul 20, 2021

Copy link
Copy Markdown
Member

We lost some coverage over time on the docs/, you can see more about it here: #1904.

@tiangolo

Copy link
Copy Markdown
Member

I imagine this is done, right? You are not planning anything else with this, right?

I'll close this one, but if you wanted to work more on something around this, please create a new/fresh PR, thanks! 🍰

@tiangolo tiangolo closed this Aug 24, 2022
@graingert

graingert commented Aug 24, 2022

Copy link
Copy Markdown
Contributor Author

I imagine this is done, right? You are not planning anything else with this, right?

I'll close this one, but if you wanted to work more on something around this, please create a new/fresh PR, thanks! 🍰

I think this was ready to merge after the conflicts get fixed, not sure why you closed it

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.

3 participants