📝 Docs preview for commit6a7b9f4 at:https://6159f015663db637a7488e2c--fastapi.netlify.app

Awesome, thank you@ghandic! 🚀

I just updated the main page as well, the README is generated from it, so now tests should pass. Thanks for your contribution! 🍰

I also updated the Coverage upload, it's needed in PRs too to be able to measure their coverage changes.

📝 Docs preview for commit7e3ccf1 at:https://6159f1281218000f3b70ed2c--fastapi.netlify.app

Codecov Report

Merging#3979 (c8008e6) intomaster (1b6350a) willnot change coverage.
The diff coverage isn/a.

@@            Coverage Diff            @@##            master     #3979   +/-   ##=========================================  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 byCodecov. Last update1b6350a...c8008e6. Read thecomment docs.

📝 Docs preview for commitc8008e6 at:https://6159f203bb69fce44748d893--fastapi.netlify.app

You'll be measuring the PR coverage and displaying that on the main page though. I did have a look to see if there was some filtering that could be done for that badge but couldn't see anything at a first pass.

I'll continue to look 👍🏻

We could add flags conditional of whether it is a PR or a push and then filter by that flag for the badge. I'll make another PR today to fix, that way you'll keep the codecov run for PR to check but also have the coverage % accurate on the readme

https://community.codecov.com/t/badges-for-flags/270/5

@tiangolo see PR -#3982