Skip to content

Comments

✏️ Clean up documentation in install.md file#1606

Merged
svlandeg merged 3 commits intofastapi:masterfrom
Johandielangman:docs/typo-in-install
Feb 23, 2026
Merged

✏️ Clean up documentation in install.md file#1606
svlandeg merged 3 commits intofastapi:masterfrom
Johandielangman:docs/typo-in-install

Conversation

@Johandielangman
Copy link
Contributor

@Johandielangman Johandielangman commented Feb 23, 2026

I see there is a "///" markup at the bottom of https://typer.tiangolo.com/tutorial/install/

Not sure if this is a typo, so I created a PR. Apologies if this has been raised before and closed. Love this project!

@github-actions github-actions bot added the docs Improvements or additions to documentation label Feb 23, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 23, 2026

📝 Docs preview

Last commit e835809 at: https://6d4bd04a.typertiangolo.pages.dev

Modified Pages

@Johandielangman
Copy link
Contributor Author

📝 Docs preview

Last commit 45eead0 at: https://373082e0.typertiangolo.pages.dev

Modified Pages

This is amazing!

@svlandeg svlandeg changed the title docs: remove "///" from the bottom of the install.md file ✏️ Remove "///" from the bottom of the install.md file Feb 23, 2026
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.

Nice catch, thank you!

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.

There's actually slightly more going on here. #1522 should have removed the whole note section, from the first /// to the last ///. Because now, pip install typer is there twice on the page.

@svlandeg svlandeg changed the title ✏️ Remove "///" from the bottom of the install.md file ✏️ Clean up documentation in install.md file Feb 23, 2026
@svlandeg svlandeg merged commit 2e78834 into fastapi:master Feb 23, 2026
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants