-
-
Notifications
You must be signed in to change notification settings - Fork 31.8k
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
Migrated SABnzbd sensor to asyncio and switched to pypi library #12290
Conversation
Ok to merge when tests pass. |
|
Sorry guys, I'm not sure why I'm getting those errors. The package is on PyPI and everything is working locally. Am I missing something somewhere? |
You have |
I think |
@andrey-git thank you for finding this. Man, what a bonehead mistake. I went ahead and just moved the classes into |
Build log now showing
and other similar errors. Is this #12283 related? |
Yeah, it's super flaky 😢 |
Description:
Created asyncio version of SABnzbd API library and published to PyPI. Migrated sabnzbd sensor to use asyncio and new pypi library.
Checklist:
If the code communicates with devices, web services, or third-party tools:
tox
run successfully. Your PR cannot be merged unless tests passREQUIREMENTS
variable (example).requirements_all.txt
by runningscript/gen_requirements_all.py
.