Python docs #446
Python docs #446
Conversation
Co-authored-by: Francesco Guardiani <[email protected]>
…d Python awakeables
Deploying documentation with
|
| Latest commit: |
7d799ed
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://2ac7b843.documentation-beg.pages.dev |
| Branch Preview URL: | https://python-docs.documentation-beg.pages.dev |
# Conflicts: # docs/develop/go/overview.mdx # docs/overview.mdx
| You can run them as follows: | ||
|
|
||
| ```shell | ||
| python -m hypercorn --config hypercorn-config.toml -b localhost:9080 example:app |
There was a problem hiding this comment.
Same comment as #447 (comment), i don't think you need to specify the bind address here
There was a problem hiding this comment.
I think it's better to do so though since all our docs are written with the assumption that the service runs on 9080... Otherwise users will just have to look that up. I think it makes the experience just a bit more fluent.
There was a problem hiding this comment.
then how about you remove the hypercorn config? at least in the quickstart, it adds some noise that is not needed.
There was a problem hiding this comment.
Or can we add it to the hypercorn config? Then we can remove it from the other commands
There was a problem hiding this comment.
But the bind address is in the hypercorn config: https://github.com/restatedev/examples/blob/main/templates/python/hypercorn-config.toml#L1 that's why specifying that -b is redundant
Still to do in follow-up PRs: Quickstart, concept pages