Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

don't change state after application started #437

Merged
merged 2 commits into from
Jan 4, 2022

Conversation

blankdots
Copy link
Contributor

@blankdots blankdots commented Dec 30, 2021

Description

corresponds to: https://github.com/aio-libs/aiohttp/blob/master/aiohttp/web_app.py#L146 and https://docs.aiohttp.org/en/stable/web_advanced.html#background-tasks

Related issues

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Changes Made

  1. changing state after application is not possible any more adding the states to on_startup for the web.Application
  2. adjust unit tests
  3. remove tox envs that are not used for py37 and py38

Testing

  • Tests do not apply

Mentions

Pending Testing ... to see if it behaves as it should

@blankdots blankdots self-assigned this Dec 30, 2021
@blankdots blankdots force-pushed the bugfix/chaing-state-of-server branch from c5e59e5 to 5513f8a Compare December 30, 2021 05:38
@blankdots blankdots changed the title don't change sate after application started don't change state after application started Dec 30, 2021
@blankdots blankdots force-pushed the bugfix/chaing-state-of-server branch from 5513f8a to be1b131 Compare December 30, 2021 05:41
@blankdots blankdots marked this pull request as ready for review December 30, 2021 06:58
swift_browser_ui/ui/server.py Outdated Show resolved Hide resolved
tests/request/test_server.py Outdated Show resolved Hide resolved
@blankdots blankdots force-pushed the bugfix/chaing-state-of-server branch 2 times, most recently from 46b4fe9 to 2008d9e Compare December 31, 2021 06:26
@blankdots blankdots force-pushed the bugfix/chaing-state-of-server branch from 2008d9e to 7cbdeba Compare December 31, 2021 06:36
@blankdots blankdots requested a review from csc-felipe December 31, 2021 06:58
@blankdots blankdots merged commit 340a733 into devel Jan 4, 2022
@blankdots blankdots deleted the bugfix/chaing-state-of-server branch January 4, 2022 12:06
@sampsapenna sampsapenna mentioned this pull request Jan 12, 2022
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants