Skip to content

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