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

Skip to content

✏ Update Hypercorn link, now pointing to GitHub#5346

Merged
tiangolo merged 3 commits into
fastapi:masterfrom
baconfield:patch-1
Sep 4, 2022
Merged

✏ Update Hypercorn link, now pointing to GitHub#5346
tiangolo merged 3 commits into
fastapi:masterfrom
baconfield:patch-1

Conversation

@baconfield

Copy link
Copy Markdown
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

github-actions Bot commented Sep 4, 2022

Copy link
Copy Markdown
Contributor

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

@iudeen iudeen left a comment

Copy link
Copy Markdown
Contributor

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

Comment thread docs/en/docs/index.md
</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
Copy Markdown
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
Copy Markdown
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

github-actions Bot commented Sep 4, 2022

Copy link
Copy Markdown
Contributor

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

@tiangolo

tiangolo commented Sep 4, 2022

Copy link
Copy Markdown
Member

Nice, thanks @baconfield! 🍰

And thanks for the help @iudeen!

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

codecov Bot commented Sep 4, 2022

Copy link
Copy Markdown

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 fastapi:master Sep 4, 2022
kittydoor added a commit to kittydoor/fastapi that referenced this pull request Jun 20, 2024
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.
kittydoor added a commit to kittydoor/fastapi that referenced this pull request Jun 20, 2024
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.
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