-
Notifications
You must be signed in to change notification settings - Fork 13
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
feat: start options to support debugpy #367
base: main
Are you sure you want to change the base?
Conversation
Note(s) for PR Auther:
Note(s) for PR Reviewer(s):
|
src/main.py
Outdated
|
||
|
||
if __name__ == "__main__": | ||
uvicorn.run(app, host="localhost", port=8000) |
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.
do you need this to run the app with python src/main.py
?
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.
yes, if I want to debug it, I do need this, as the referenced docs of FastAPI shows.
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.
Somehow fastapi
cli is the recommended way in the docs https://fastapi.tiangolo.com/#standard-dependencies.
Can you share the doc that you read? I really used uvicorn in the main and there were some issue for me.
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.
Description
Changes proposed in this pull request:
main.py
to allow debugg viadebugpy
, the DAP implementation for Python that is supportet by NVim, as mentioned in the official docs of FastAPIRelated issue(s)