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

Skip to content

✏ Rewording in docs/en/docs/tutorial/debugging.md#9581

Merged
tiangolo merged 3 commits into
fastapi:masterfrom
ivan-abc:en/docs/tutorial/debugging
Jun 22, 2023
Merged

✏ Rewording in docs/en/docs/tutorial/debugging.md#9581
tiangolo merged 3 commits into
fastapi:masterfrom
ivan-abc:en/docs/tutorial/debugging

Conversation

@ivan-abc

Copy link
Copy Markdown
Contributor

A suggestion to change some words in __name__ == "__main__" section of the debugging.md to simplify some expressions.

And also replaced the word "automatic" in order not to mix up with automatic variable definition 🤔 (as I understand it has another meaning in this context).

@github-actions

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit d287ed1 at: https://64731d81bf49b716013a6d0a--fastapi.netlify.app

@tiangolo

Copy link
Copy Markdown
Member

Thanks!

I actually prefer the original words as the variables are not being set by the user, or by FastAPI or anything else, they just "have the value". Of course, this becomes a lot dependent on personal preference, but anyway, I prefer the previous wording.

I had never heard the term "automatic variable", I guess that comes from C++ and Java, maybe. But anyway, I updated it to not use that exact term.

Thanks! ☕

@tiangolo tiangolo changed the title ✏ Rewordings in docs/en/docs/tutorial/debugging.md ✏ Rewording in docs/en/docs/tutorial/debugging.md Jun 22, 2023
@tiangolo tiangolo enabled auto-merge (squash) June 22, 2023 17:00
@tiangolo tiangolo merged commit 68ce5b3 into fastapi:master Jun 22, 2023
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.

2 participants