-
-
Notifications
You must be signed in to change notification settings - Fork 6.9k
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
✏ Update Hypercorn link, now pointing to GitHub #5346
Conversation
Updated Uvicorn link from gitlab to github to reflect current hosting https://gitlab.com/pgjones/hypercorn/-/commit/65e539a9dc762cb19f4907038fe6d490dac909ff
📝 Docs preview for commit b74135e at: https://63146b95ea029449b4f616d7--fastapi.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The title of the PR should be "Update Hypercorn link" not Uvicorn....
@@ -128,7 +128,7 @@ $ pip install fastapi | |||
|
|||
</div> | |||
|
|||
You will also need an ASGI server, for production such as <a href="https://www.uvicorn.org" class="external-link" target="_blank">Uvicorn</a> or <a href="https://gitlab.com/pgjones/hypercorn" class="external-link" target="_blank">Hypercorn</a>. | |||
You will also need an ASGI server, for production such as <a href="https://www.uvicorn.org" class="external-link" target="_blank">Uvicorn</a> or <a href="https://github.com/pgjones/hypercorn" class="external-link" target="_blank">Hypercorn</a>. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You will also need an ASGI server, for production such as <a href="https://www.uvicorn.org" class="external-link" target="_blank">Uvicorn</a> or <a href="https://github.com/pgjones/hypercorn" class="external-link" target="_blank">Hypercorn</a>. | |
You will also need an ASGI server, for production such as <a href="https://www.uvicorn.org" class="external-link" target="_blank">Uvicorn</a> or <a href="https://hypercorn.readthedocs.io/" class="external-link" target="_blank">Hypercorn</a>. |
Sorry about that, was half asleep there |
📝 Docs preview for commit 6aea685 at: https://6314b832892a157d58638265--fastapi.netlify.app |
Nice, thanks @baconfield! 🍰 And thanks for the help @iudeen! |
Codecov Report
@@ Coverage Diff @@
## master #5346 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 539 539
Lines 13902 13902
=========================================
Hits 13902 13902 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
PR fastapi#5346 seems to have partially done this already, however a lot of references are still outdated in the docs. This commit updates all remainders.
PR fastapi#5346 seems to have partially done this already, however a lot of references are still outdated in the docs. This commit updates all remainders.
Updated Uvicorn link from gitlab to github to reflect current hosting
https://gitlab.com/pgjones/hypercorn/-/commit/65e539a9dc762cb19f4907038fe6d490dac909ff