8000 ✏ Fix typo in Benchmarks by AlejoAsd · Pull Request #1995 · 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 Benchmarks #1995

Merged
merged 1 commit into from
Nov 5, 2020

Conversation

AlejoAsd
Copy link
Contributor
@AlejoAsd AlejoAsd commented Sep 1, 2020

Fixed grammatical error in the benchmarks documentation.

@codecov
Copy link
codecov bot commented Sep 1, 2020

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #1995   +/-   ##
=========================================
  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 e77ea63...dd1eeca. Read the comment docs.

@github-actions
Copy link
Contributor
github-actions bot commented Sep 1, 2020

📝 Docs preview for commit dd1eeca at: https://5f4dca41ef6c12635ce05ab4--fastapi.netlify.app

@AlejoAsd
Copy link
Contributor Author
AlejoAsd commented Sep 9, 2020

Should this be merged?

@tiangolo tiangolo changed the title Fix benchmarks documentation grammar error. ✏ Fix typo in Benchmarks Nov 5, 2020
@tiangolo tiangolo merged commit e096464 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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0