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

Update path-params.md#256

Merged
tiangolo merged 3 commits intofastapi:masterfrom
wshayes:patch-3
May 29, 2019
Merged

Update path-params.md#256
tiangolo merged 3 commits intofastapi:masterfrom
wshayes:patch-3

Conversation

@wshayes
Copy link
Contributor

@wshayeswshayes commentedMay 24, 2019
edited
Loading

Add path parameter converters (e.g. starlette urlconverters) documentation.

@codecov
Copy link

codecovbot commentedMay 24, 2019

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff          @@##           master   #256   +/-   ##=====================================  Coverage     100%   100%           =====================================  Files         177    177             Lines        4327   4327           =====================================  Hits         4327   4327

Continue to review full report at Codecov.

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

@codecov
Copy link

codecovbot commentedMay 24, 2019
edited
Loading

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@##           master   #256    +/-   ##======================================  Coverage     100%   100%            ======================================  Files         177    194    +17       Lines        4327   4723   +396     ======================================+ Hits         4327   4723   +396
Impacted FilesCoverage Δ
fastapi/dependencies/utils.py100% <0%> (ø)⬆️
fastapi/dependencies/models.py100% <0%> (ø)⬆️
fastapi/applications.py100% <0%> (ø)⬆️
fastapi/encoders.py100% <0%> (ø)⬆️
fastapi/routing.py100% <0%> (ø)⬆️
tests/test_ws_router.py100% <0%> (ø)⬆️
...t_tutorial/test_response_model/test_tutorial004.py100% <0%> (ø)
docs/src/body_updates/tutorial001.py100% <0%> (ø)
docs/src/response_model/tutorial006.py100% <0%> (ø)
docs/src/response_model/tutorial005.py100% <0%> (ø)
... and13 more

Continue to review full report at Codecov.

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

@tiangolo
Copy link
Member

Thanks! I'll check it soon.

@tiangolotiangolo merged commitd544bdf intofastapi:masterMay 29, 2019
@tiangolo
Copy link
Member

Thank you@wshayes ! Checking this I realized that paths in path operations could be improved to not require aPath(...).

I fixed/improved it at:#272

Then I updated the docs here a bit, to include a source file in the docs instead of having the Python code copied. That allows the source code to be tested. That way we are sure that the code in the docs runs correctly, and helps us ensure that features are properly documented.

Now it's merged. 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

@wshayes@tiangolo

Comments


[8]ページ先頭

©2009-2026 Movatter.jp