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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃寪 Improve the docs website #4626

Closed
wants to merge 1 commit into from
Closed

Conversation

hanxiao
Copy link

@hanxiao hanxiao commented Feb 28, 2022

As requested by Sebasti谩n, I added this feature to the docs.

You can find the preview here: https://fastapi-qabot-integration.netlify.app/

As requested by Sebasti谩n, I added this feature to the docs.
You can find the preview here: https://fastapi-qabot-integration.netlify.app/
@codecov
Copy link

codecov bot commented Mar 5, 2022

Codecov Report

Merging #4626 (01e0d0c) into master (440d2d2) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #4626   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          526       526           
  Lines        13324     13324           
=========================================
  Hits         13324     13324           

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 440d2d2...01e0d0c. Read the comment docs.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2022

馃摑 Docs preview for commit 01e0d0c at: https://6223ae8833f911629ebbc34c--fastapi.netlify.app

@tiangolo
Copy link
Owner

tiangolo commented Mar 5, 2022

Awesome, thank you! I included your commits in this PR #4655 and added a couple of tweaks on top. I didn't do it in this same PR as I don't have write access. As this is now merged in the other PR, I'll close this one.

Thanks for this! 馃殌

@tiangolo tiangolo closed this Mar 5, 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.

None yet

3 participants