Skip to content

Use a context manager for simplified auto-notification of users #2390

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.

Sign up for GitHub

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

Jump to bottom
Closed
sanderegg opened this issue Jun 18, 2021 · 1 comment
Closed

Use a context manager for simplified auto-notification of users #2390

sanderegg opened this issue Jun 18, 2021 · 1 comment
Assignees
@sanderegg
Labels
a:webserver issue related to the webserver service t:enhancement Improvement or request on an existing feature
Milestone
Marmoset

Comments

@sanderegg
Copy link
Member

sanderegg commented Jun 18, 2021 •
edited
Loading

Use an async context manager when python 3.8 gets in in projects_api.py

import sys
if sys.version_info >= (3, 8):
    raise Exception("use below implementation in py3.8+")

this way you won't miss it

Originally posted by @GitHK in #2383 (comment)
@sanderegg sanderegg self-assigned this Jun 18, 2021
@sanderegg sanderegg added a:webserver issue related to the webserver service t:enhancement Improvement or request on an existing feature labels Jun 18, 2021
@sanderegg sanderegg added this to the Marmoset milestone Jun 18, 2021
@sanderegg sanderegg changed the title use the lock with notification Use a context manager for simplified auto-notification of users Jun 18, 2021
@sanderegg
Copy link
Member Author

in use now.

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

@sanderegg sanderegg

Labels
a:webserver issue related to the webserver service t:enhancement Improvement or request on an existing feature
Projects
None yet
Milestone
Marmoset
Development

No branches or pull requests
1 participant
@sanderegg