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

Added state to FastAPI and up starlette to 0.12.9#502

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
euri10 wants to merge1 commit intofastapi:masterfromeuri10:up_starlette
Closed

Added state to FastAPI and up starlette to 0.12.9#502

euri10 wants to merge1 commit intofastapi:masterfromeuri10:up_starlette

Conversation

@euri10
Copy link
Contributor

and get access to applicationstate for 💲 💰

zamiramir, b1-luettje, faheem-nadeem, and krnr reacted with thumbs up emoji
@codecov
Copy link

codecovbot commentedSep 4, 2019•
edited
Loading

Codecov Report

Merging#502 intomaster willnot change coverage.
The diff coverage is100%.

Impacted file tree graph

@@          Coverage Diff          @@##           master   #502   +/-   ##=====================================  Coverage     100%   100%           =====================================  Files         251    252    +1       Lines        5988   5948   -40     =====================================- Hits         5988   5948   -40
Impacted FilesCoverage Δ
tests/test_state.py100% <100%> (ø)
fastapi/applications.py100% <100%> (ø)⬆️
fastapi/__init__.py100% <100%> (ø)⬆️
...est_additional_responses_custom_validationerror.py100% <0%> (ø)⬆️
docs/src/app_testing/tutorial003.py100% <0%> (ø)⬆️
...t_query_params_str_validations/test_tutorial001.py100% <0%> (ø)⬆️
tests/test_custom_swagger_ui_redirect.py100% <0%> (ø)⬆️
docs/src/first_steps/tutorial001.py100% <0%> (ø)⬆️
tests/test_additional_responses_router.py100% <0%> (ø)⬆️
docs/src/application_configuration/tutorial001.py100% <0%> (ø)⬆️
... and64 more

Continue to review full report at Codecov.

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

Pushes Starlette to 0.12.9Added State to FastAPI to better mimic Starlette 0.12.9 behaviourAdded tests, one shamelessly taken from Starlette, the other testing state behaviour across startup / shutdown events
@euri10euri10 changed the titleUp starlette to 0.12.9Added state to FastAPI and up starlette to 0.12.9Sep 10, 2019
@tiangolo
Copy link
Member

Thanks@euri10 !

It seems the Git history of this branch got into a weird state and is showing some extra changes that are already there and create conflicts. I also had to update the range, as Starlette'sState is not available in0.11.1.

So I just merged#593 that includes these changes. But thanks for your contribution! 🍰

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
[8]ページ先頭
©2009-2025 Movatter.jp