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

extra starlette options docs#25

Merged
tiangolo merged 6 commits intofastapi:masterfrom
euri10:starlette_options_docs
Feb 16, 2019
Merged

extra starlette options docs#25
tiangolo merged 6 commits intofastapi:masterfrom
euri10:starlette_options_docs

Conversation

@euri10
Copy link
Contributor

Added a hint in docs on how to access the starlette Request object after you showed me how to in#19 !

@codecov
Copy link

codecovbot commentedFeb 8, 2019•
edited
Loading

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@##           master    #25    +/-   ##======================================  Coverage     100%   100%            ======================================  Files          95     99     +4       Lines        2484   2590   +106     ======================================+ Hits         2484   2590   +106
Impacted FilesCoverage Δ
tests/test_jsonable_encoder.py100% <0%> (ø)⬆️
fastapi/openapi/utils.py100% <0%> (ø)⬆️
fastapi/applications.py100% <0%> (ø)⬆️
fastapi/encoders.py100% <0%> (ø)⬆️
tests/test_datetime.py
docs/src/sql_databases/tutorial001.py100% <0%> (ø)
...tutorial/test_sub_applications/test_tutorial001.py100% <0%> (ø)
tests/test_datetime_custom_encoder.py100% <0%> (ø)
...st_tutorial/test_sql_databases/test_tutorial001.py100% <0%> (ø)
docs/src/sub_applications/tutorial001.py100% <0%> (ø)

Continue to review full report at Codecov.

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

@tiangolotiangolo merged commitfebf8e7 intofastapi:masterFeb 16, 2019
@tiangolo
Copy link
Member

tiangolo commentedFeb 16, 2019•
edited
Loading

Good idea! Thanks for your work! 🎉 🌮

I updated it a bit to simplify it and make it more "focused" on the basics of theRequest object, independent of database parts.

Merged! ✔️ 🚀

The new docs are here:https://fastapi.tiangolo.com/tutorial/using-request-directly/

euri10 reacted with thumbs up emoji

@euri10
Copy link
ContributorAuthor

This is indeed way better than what i wrote, amazing as always !

tiangolo reacted with thumbs up emoji

@euri10euri10 deleted the starlette_options_docs branchFebruary 18, 2019 20:07
@tiangolo
Copy link
Member

Thanks! 😊 🎉

And thanks for starting it 👍

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

@euri10@tiangolo

Comments

[8]ページ先頭
©2009-2026 Movatter.jp