Skip to content

📝 Update includes in docs/en/docs/advanced/generate-clients.md#12642

Merged
alejsdev merged 6 commits into
fastapi:masterfrom
AyushSinghal1794:AyushSinghal1794-patch-10-1
Nov 9, 2024
Merged

📝 Update includes in docs/en/docs/advanced/generate-clients.md#12642
alejsdev merged 6 commits into
fastapi:masterfrom
AyushSinghal1794:AyushSinghal1794-patch-10-1

Conversation

@AyushSinghal1794

Copy link
Copy Markdown
Contributor

Update code snippets to align with new formatting guidelines from PR #12580

Update code snippets to align with new formatting guidelines from PR fastapi#12580
@github-actions github-actions Bot added the docs Documentation about how to use FastAPI label Oct 28, 2024
@github-actions

Copy link
Copy Markdown
Contributor

@AyushSinghal1794 AyushSinghal1794 changed the title Update includes in docs/en/docs/advanced/generate-clients.md Update includes in docs/en/docs/advanced/generate-clients.md Oct 29, 2024
@AyushSinghal1794

AyushSinghal1794 commented Oct 29, 2024

Copy link
Copy Markdown
Contributor Author

Hi @alejsdev, thanks for helping me out, I have implemented the suggested changes in this new PR as suggested in #12580, can you please review it.
Thanks for your efforts

@github-actions

github-actions Bot commented Nov 9, 2024

Copy link
Copy Markdown
Contributor

@alejsdev

alejsdev commented Nov 9, 2024

Copy link
Copy Markdown
Member

Thanks @AyushSinghal1794
I had to modify the last one (it's a special case)
Thanks for your help! 🎉

@alejsdev alejsdev changed the title Update includes in docs/en/docs/advanced/generate-clients.md 📝 Update includes in docs/en/docs/advanced/generate-clients.md Nov 9, 2024
@alejsdev alejsdev enabled auto-merge (squash) November 9, 2024 16:02
@github-actions

github-actions Bot commented Nov 9, 2024

Copy link
Copy Markdown
Contributor

@alejsdev alejsdev disabled auto-merge November 9, 2024 16:40
@github-actions

github-actions Bot commented Nov 9, 2024

Copy link
Copy Markdown
Contributor

@alejsdev alejsdev merged commit ac4956d into fastapi:master Nov 9, 2024
@AyushSinghal1794

Copy link
Copy Markdown
Contributor Author

Thank you so much, @alejsdev I appreciate you handling that special case, and I'm glad to have contributed. Looking forward to helping out more in the future! 🎉😊

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.

2 participants