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

Optimization and refinements done for Home-Assistant #13

Merged
merged 14 commits into from
Feb 10, 2018

Conversation

jabesq
Copy link
Contributor

@jabesq jabesq commented Feb 9, 2018

This PR contains some optimization done for Home-Assistant

Question: Is there a plan to send this API on Pypi so it will be easier to deploy and manage ?

turbokongen and others added 14 commits April 7, 2017 14:13
Return raw response to let consuming code decide what to do to the data.
Signed-off-by: Hugo DUPRAS <[email protected]>
… mode & navigationMode to startCleaning

(cherry picked from commit dedf9c7)
A few changes before merging into main branch:
- Create UnsupportedDevice exception class, to simplify catching exception
  in user code.
- Create read-only properties for available_services and
  service_version.
- Check that service version is supported once (in __init__), to avoid
  code duplication.
- Renamed a few variables for PEP 8 compliance.
- Bumped version

(cherry picked from commit 43d9f41)
Fix header format

(cherry picked from commit 0fc58bf)
@stianaske stianaske merged commit d3906d0 into stianaske:master Feb 10, 2018
@stianaske
Copy link
Owner

Looks good! Thanks!
I did some preparations for adding it to Pypi, but never actually launched id. I'll look into it.

@andrey-git
Copy link

@stianaske please upload to pypi if possible.

I think it should be as easy as

pip3 install twine
python3 setup.py sdist
twine upload dist/*

after you create pypi user.

@stianaske
Copy link
Owner

@andrey-git version 0.0.7 has been uploaded to pypi. Appologies for the slow response.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants