Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Comments

📝 Fix code consistency in examples for Tutorial - User Guide - Path Parameters#2158

Merged
tiangolo merged 4 commits intofastapi:masterfrom
nukopy:fix-typos-in-docs_src
Nov 5, 2020
Merged

📝 Fix code consistency in examples for Tutorial - User Guide - Path Parameters#2158
tiangolo merged 4 commits intofastapi:masterfrom
nukopy:fix-typos-in-docs_src

Conversation

@nukopy
Copy link

@nukopynukopy commentedOct 10, 2020
edited
Loading

I found typos in the chapter "Path Parameters" in "Tutorial - User Guide".

What

Endpoint's pathmodel inexample code should be a plural form,models.

I fixed below:

  • docs_src/path_params/tutoial005.py
  • image associated with this example:docs/en/docs/img/tutorial/image03.png

@Kludex
Copy link
Member

The pipeline is failing because the tests are calling/model instead of/models.

nukopy reacted with thumbs up emoji

@codecov
Copy link

codecovbot commentedOct 10, 2020
edited
Loading

Codecov Report

Merging#2158 intomaster willnot change coverage.
The diff coverage isn/a.

Impacted file tree graph

@@            Coverage Diff            @@##            master     #2158   +/-   ##=========================================  Coverage   100.00%   100.00%           =========================================  Files          239       239             Lines         7079      7079           =========================================  Hits          7079      7079
Impacted FilesCoverage Δ
...test_tutorial/test_path_params/test_tutorial005.py100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last updatee77ea63...e131337. Read thecomment docs.

@nukopy
Copy link
Author

Thanks. I fixed tests.

@nukopynukopy changed the titleWIP: Fix typos indocs_src in "Tutorial - User Guide"Fix typos indocs_src in "Tutorial - User Guide"Oct 10, 2020
@github-actions
Copy link
Contributor

📝 Docs preview for commite131337 at:https://5f8224071a19b43513081953--fastapi.netlify.app

@tiangolotiangolo changed the titleFix typos indocs_src in "Tutorial - User Guide"📝 Fix code consistency in examples for Tutorial - User Guide - Path ParametersNov 5, 2020
@tiangolotiangolo merged commit22a5960 intofastapi:masterNov 5, 2020
@tiangolo
Copy link
Member

Nice! Good catch, thank you! 🚀 🎉

Thanks@Kludex for the help! 💪 🍰

ycd and nukopy reacted with hooray emojiycd and nukopy reacted with heart emojiycd and nukopy reacted with rocket emoji

@nukopynukopy deleted the fix-typos-in-docs_src branchNovember 6, 2020 10:40
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nukopy@Kludex@tiangolo

[8]ページ先頭

©2009-2026 Movatter.jp