-
-
Notifications
You must be signed in to change notification settings - Fork 8.5k
📝 Update en/docs/tutorial/debugging.md
#10590
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 en/docs/tutorial/debugging.md
#10590
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
* Full translation * In sync with fastapi#10590 * Google Translate used * Manually checked * Checked for errors using LanguageTool
This comment was marked as outdated.
This comment was marked as outdated.
When originally rewriting the text I modified the German translation of the original English text and, when done, I translated everything back to English. Some things got lost while doing that, which this commit fixes. Also undo the changes of the previous commit.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
Thanks for the review, @alejsdev |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
📝 Docs preview for commit ad3fbd8 at: https://83dae283.fastapitiangolo.pages.dev Modified Pages |
YuriiMotov
left a comment
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.
@nilslindemann, thanks for your efforts!
I think there are a couple of things that need to be improved. 🧐
Would you like to continue working on this?
|
|
||
| * Next, choose "FastAPI" from the options. | ||
|
|
||
| * In the subsequent dropdown, enter the name of your application file, here `myapp.py` (This is only necessary because VS Code did not find a `main.py` file). |
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.
Looks like this has been changed. I tried to follow steps and it didn't ask me to provide file name. It showed launch.json with default main.py right after I chose "FastAPI" on previous step.
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.
Thanks, Yuri, I currently don't have time for this. Very probable that this is meanwhile outdated again. After all, the PR is nearly two years old. I will close this issue for now, and may create a new one when there is time.
| ```Python hl_lines="1 15" | ||
| {!../../../docs_src/debugging/tutorial002.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.
| ```Python hl_lines="1 15" | |
| {!../../../docs_src/debugging/tutorial002.py!} | |
| ``` | |
| {* ../../docs_src/debugging/tutorial002.py hl[1,15] *} |
|
|
||
| ## PyCharm | ||
|
|
||
| PyCharm does not currently have a debug configuration specifically for FastAPI applications, so we will run the `uvicorn` server directly from our `myapp.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.
|
Closing this as it is outdated, as @YuriiMotov pointed out. Will maybe recreate at a later time. Also, Party! I don't have any open PRs in this Repo anymore. It was a fun time, @tiangolo, @alejsdev and all others I have met. See you again. |
|
@nilslindemann, thank you for your contribution! |
This chapter was outdated:
uvicornfrom themyapp.pyis not necessary.I kept the
__name__ == "__main__"part, but rewrote the rest.Feel free to modify as you want.
#10591 is a translation of this.