Skip to content

Comments

✏️ Fix typo in README installation instructions#13011

Merged
alejsdev merged 4 commits intofastapi:masterfrom
dave-hay:fix-readme-grammar
Dec 9, 2024
Merged

✏️ Fix typo in README installation instructions#13011
alejsdev merged 4 commits intofastapi:masterfrom
dave-hay:fix-readme-grammar

Conversation

@dave-hay
Copy link
Contributor

Fixed a missing word in the installation instructions:

Before: "it comes the standard group"
After: "it comes with the standard group"

@svlandeg svlandeg added the feature New feature or request label Dec 4, 2024
@github-actions github-actions bot added the docs Documentation about how to use FastAPI label Dec 4, 2024
@svlandeg svlandeg changed the title Fix grammar: Add missing 'with' in README installation instructions ✏️ Fix typo in README installation instructions Dec 4, 2024
@svlandeg svlandeg removed the feature New feature or request label Dec 4, 2024
Copy link
Member

@svlandeg svlandeg left a comment

Choose a reason for hiding this comment

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

Thanks, good catch! This was also wrong in the original index.md file, so I added that fix as well.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2024

📝 Docs preview for commit f205c16 at: https://c7119fce.fastapitiangolo.pages.dev

Modified Pages

@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2024

📝 Docs preview for commit 6f431e3 at: https://780ead9a.fastapitiangolo.pages.dev

Modified Pages

@alejsdev alejsdev merged commit 9e19613 into fastapi:master Dec 9, 2024
@alejsdev
Copy link
Member

alejsdev commented Dec 9, 2024

Good catch! Thank you @dave-hay 🔍
And thanks for the update @svlandeg 😸

@dave-hay dave-hay deleted the fix-readme-grammar branch December 11, 2024 02:07
s-rigaud pushed a commit to s-rigaud/fastapi that referenced this pull request Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation about how to use FastAPI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants