-
-
Notifications
You must be signed in to change notification settings - Fork 543
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
Add consumable status to viomi vacuum #805
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thanks for the PR! 👍 This can be merged after the checks are passing.
@rytilahti I fixed the tests in my changes, but the tests are broken in master. What should I do ? |
@titilambert oops, I didn't notice prior merging that the lastly merged PR broke linting.. I think the best way to proceed is to incorporate the linting fixes to this PR, so please run |
OK ! I just fixed the lint, the code coverage decreased, is it fine ? |
Decrease in code coverage is not that big issue, although it'd be great to have tests for these parts. Let's get this merged so that the PR I linked can be rebased on top of this, thanks again for the PR! |
* Add consumable status to viomi * Fix Lint * Fix lint
Tested with:
Model: viomi.vacuum.v7
Hardware version: Linux
Firmware version: 3.5.3_0047