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

Skip to content

🎨 Fix syntax highlighting in docs for OpenAPI Callbacks#4368

Merged
tiangolo merged 2 commits into
fastapi:masterfrom
xncbf:patch-1
Aug 26, 2022
Merged

🎨 Fix syntax highlighting in docs for OpenAPI Callbacks#4368
tiangolo merged 2 commits into
fastapi:masterfrom
xncbf:patch-1

Conversation

@xncbf

@xncbf xncbf commented Jan 5, 2022

Copy link
Copy Markdown
Contributor

image

Code line highlight line does not match description like above.

@xncbf

xncbf commented Jan 20, 2022

Copy link
Copy Markdown
Contributor Author

please review

@codecov

codecov Bot commented Aug 26, 2022

Copy link
Copy Markdown

Codecov Report

Merging #4368 (81d1e35) into master (f928f33) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master     #4368   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          535       535           
  Lines        13830     13830           
=========================================
  Hits         13830     13830           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@github-actions

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 81d1e35 at: https://6308982fcf308e71fce74ef7--fastapi.netlify.app

@xncbf

xncbf commented Aug 26, 2022

Copy link
Copy Markdown
Contributor Author

Looks good :)

@tiangolo tiangolo changed the title Code line highlight line does not match description 🎨 Fix syntax highlighting in docs for OpenAPI Callbacks Aug 26, 2022
@tiangolo

Copy link
Copy Markdown
Member

Good catch, thank you! 🚀 🎉

@tiangolo tiangolo merged commit bb53d0b into fastapi:master Aug 26, 2022
@xncbf xncbf deleted the patch-1 branch August 26, 2022 13:24
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