You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is the error we're seeing during the testing of the above PR:
==================================== ERRORS ====================================
_________ ERROR collecting tests/components/vilfo/test_config_flow.py __________
ImportError while importing test module '/home/runner/work/core/core/tests/components/vilfo/test_config_flow.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/opt/hostedtoolcache/Python/3.11.8/x64/lib/python3.11/importlib/__init__.py:126: in import_module
return _bootstrap._gcd_import(name[level:], package, level)
tests/components/vilfo/test_config_flow.py:4: in <module>
import vilfo
venv/lib/python3.11/site-packages/vilfo/__init__.py:5: in <module>
from .client import (
venv/lib/python3.11/site-packages/vilfo/client.py:7: in <module>
from semver import compare as semver_compare
E ModuleNotFoundError: No module named 'semver'
It is caused by the plugwise-backend module removing the semver dependency in this PR, which breaks the vilfo-backend module.
Please also see frenck's comment in the Core PR.
The text was updated successfully, but these errors were encountered:
As far as I understand the issue now, the install_requires in setup.cfg are not complete, semver is missing, possibly others.
I looked at the contents of requirements.txt, there seem to be too many dependencies listed?
From what I see in client.py,ipaddress, json, requests, semver and getmac are the required dependencies?
This https://docs.python.org/3/library/ tells me ipaddress and json are standard libs, so you'll only need to add semver to setup.cfg. semver ==2.13.0 I think?
And then bump the version and release :)
Please see home-assistant/core#113245
This is the error we're seeing during the testing of the above PR:
It is caused by the plugwise-backend module removing the
semver
dependency in this PR, which breaks the vilfo-backend module.Please also see frenck's comment in the Core PR.
The text was updated successfully, but these errors were encountered: