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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

✏ Update Hypercorn link, now pointing to GitHub #5346

Merged
merged 3 commits into from Sep 4, 2022

Conversation

baconfield
Copy link
Contributor

Updated Uvicorn link from gitlab to github to reflect current hosting
https://gitlab.com/pgjones/hypercorn/-/commit/65e539a9dc762cb19f4907038fe6d490dac909ff

Updated Uvicorn link from gitlab to github to reflect current hosting
https://gitlab.com/pgjones/hypercorn/-/commit/65e539a9dc762cb19f4907038fe6d490dac909ff
@github-actions
Copy link
Contributor

github-actions bot commented Sep 4, 2022

📝 Docs preview for commit b74135e at: https://63146b95ea029449b4f616d7--fastapi.netlify.app

Copy link
Contributor

@iudeen iudeen left a 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>.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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>.

@baconfield baconfield changed the title Update uvicorn link Update Hypercorn link Sep 4, 2022
@baconfield
Copy link
Contributor Author

The title of the PR should be "Update Hypercorn link" not Uvicorn....

Sorry about that, was half asleep there

@tiangolo tiangolo changed the title Update Hypercorn link ✏ Update Hypercorn link, now pointing to GitHub Sep 4, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Sep 4, 2022

📝 Docs preview for commit 6aea685 at: https://6314b832892a157d58638265--fastapi.netlify.app

@tiangolo
Copy link
Owner

tiangolo commented Sep 4, 2022

Nice, thanks @baconfield! 🍰

And thanks for the help @iudeen!

@tiangolo tiangolo enabled auto-merge (squash) September 4, 2022 14:55
@codecov
Copy link

codecov bot commented Sep 4, 2022

Codecov Report

Merging #5346 (c6cea53) into master (9df22ab) will not change coverage.
The diff coverage is n/a.

@@            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.

@tiangolo tiangolo merged commit 0ae8d09 into tiangolo:master Sep 4, 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