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

Fix: Annotation was not supported by Python 3.8 #244

Merged
merged 1 commit into from
Oct 6, 2022
Merged

Conversation

hoh
Copy link
Member

@hoh hoh commented Oct 6, 2022

We still support Ubuntu 20.04 that ships with Python 3.8.

Solution: Use a different typing annotation depending on the version of Python, making it easier to ditch the simpler annotation when Python 3.8 goes out of support.

Fixes #243

@hoh hoh added the bug Something isn't working label Oct 6, 2022
@hoh hoh requested a review from odesenfans October 6, 2022 10:57
vm_supervisor/pubsub.py Outdated Show resolved Hide resolved
We still support Ubuntu 20.04 that ships with Python 3.8.

Solution: Use a different typing annotation depending on the version of Python, making it easier to ditch the simpler annotation when Python 3.8 goes out of support.
@hoh hoh merged commit 7c742b0 into main Oct 6, 2022
@hoh hoh deleted the hoh-fix-python-38 branch October 6, 2022 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Typing issue caused supervisor to fail starting on Ubuntu 20.04
2 participants