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

Rename a function in tutorial#1121

Merged
tiangolo merged 1 commit intofastapi:masterfrom
tmsick:improve-doc
Mar 27, 2020
Merged

Rename a function in tutorial#1121
tiangolo merged 1 commit intofastapi:masterfrom
tmsick:improve-doc

Conversation

@tmsick
Copy link
Contributor

Path Parameters - FastAPI

fromfastapiimportFastAPIapp=FastAPI()@app.get("/files/{file_path:path}")asyncdefread_user_me(file_path:str):return {"file_path":file_path}

I changed the name of the above function fromread_user_me toread_file.

@codecov
Copy link

codecovbot commentedMar 16, 2020
edited
Loading

Codecov Report

Merging#1121 intomaster willnot change coverage by%.
The diff coverage isn/a.

Impacted file tree graph

@@            Coverage Diff            @@##            master     #1121   +/-   ##=========================================  Coverage   100.00%   100.00%           =========================================  Files          223       223             Lines         6642      6642           =========================================  Hits          6642      6642
Impacted FilesCoverage Δ
...test_tutorial/test_path_params/test_tutorial004.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 update38fd8a6...92ed30c. Read thecomment docs.

@tiangolotiangolo merged commit15241b5 intofastapi:masterMar 27, 2020
@tiangolo
Copy link
Member

Ah! Good catch. Thanks! 🚀 🍰

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tmsick@tiangolo

[8]ページ先頭

©2009-2026 Movatter.jp