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

Skip to content

✏ Tweak wording to clarify docs/en/docs/project-generation.md#5930

Merged
tiangolo merged 2 commits into
fastapi:masterfrom
chandra-deb:master
Feb 9, 2023
Merged

✏ Tweak wording to clarify docs/en/docs/project-generation.md#5930
tiangolo merged 2 commits into
fastapi:masterfrom
chandra-deb:master

Conversation

@chandra-deb

Copy link
Copy Markdown
Contributor

"You can use a project generator to ...... database and first API endpoints already done for you." - In this sentence, the word "first" does not fit the context. It should be "some API".

"You can use a project generator to ...... database and first API endpoints already done for you."
In this sentence, the word "first" does not fit the context. It should be "some API".
@github-actions

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 8a58629 at: https://63d42129bda710324f3f1694--fastapi.netlify.app

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

LGTM!

@chandra-deb chandra-deb reopened this Jan 28, 2023
@chandra-deb

Copy link
Copy Markdown
Contributor Author

Sorry! I unknowingly closed my pull request. @fabioueno

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

No problem, @chandra-deb. 😉

@Ryandaydev Ryandaydev left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests pass. (Not surprising, since it's just some text changing but...)

@tiangolo tiangolo changed the title change "first API" to "some API" ✏ Tweak wording to clarify docs/en/docs/project-generation.md Feb 9, 2023
@tiangolo

tiangolo commented Feb 9, 2023

Copy link
Copy Markdown
Member

Sounds good, thanks @chandra-deb! ☕

@tiangolo tiangolo merged commit d6fb942 into fastapi:master Feb 9, 2023
axel584 pushed a commit to axel584/fastapi that referenced this pull request Mar 5, 2023
mahenzon pushed a commit to mahenzon/fastapi that referenced this pull request Mar 11, 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.

4 participants