8000 ✏️ Fix typo in Tutorial - Path Parameters by mariacamilagl · Pull Request #2231 · fastapi/fastapi · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

✏️ Fix typo in Tutorial - Path Parameters #2231

Merged
merged 1 commit into from
Nov 5, 2020

Conversation

mariacamilagl
Copy link
Collaborator

This is a super simple PR that adds backticks to a mention of ints (line 64) in the Tutorial - Path Parameters page. 😃

Not really a typo, more like a consistency check.

@codecov
Copy link
codecov bot commented Oct 23, 2020

Codecov Report

Merging #2231 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #2231   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          239       239           
  Lines         7079      7079           
=========================================
  Hits          7079      7079           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dca9cc3...da81ae5. Read the comment docs.

@github-actions
Copy link
Contributor

📝 Docs preview for commit da81ae5 at: https://5f9302ff902688011bbdd515--fastapi.netlify.app

@tiangolo tiangolo merged commit 733218f into fastapi:master Nov 5, 2020
@tiangolo
Copy link
Member
tiangolo commented Nov 5, 2020

Good catch, thanks! 🔍 ☕

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0