forked from home-assistant/core
-
Notifications
You must be signed in to change notification settings - Fork 0
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
update to latest HA #1
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Dont use DATA_ROOM_SETPOINT_OVRD in climate.opentherm_gw as it is unreliable with some thermostats. * Show new target temperature immediately until the backend notices a change * Only update target temp on the gateway if the value differs from the current target_temperature.
* Add Hass.io deCONZ discovery flow * add bridge ID * fix attribute * fix strings * Address comments * Add test * Add only instance / changed maybe later
* Fix next/previous track * Bump pychromecast * Update test, fixup
* Fix trusted networks auth provider warning message * Update auth.py
* Rebrand Cisco Spark notify to be Cisco Webex Teams * Remove property from class * Switch to use html for api * Update notify.py * Rename CONF_ROOMID to CONF_ROOM_ID * updated * Fix lint errors * Update notify.py * Update notify.py * Also validate room ID * Update notify.py * Update .coveragerc * Update notify.py
* Deal with cover assumed state * Add docs
* Add mobile_app notify platform * Requested changes * Fix incorrect param for status code * Move push_registrations to notify platform file * Trim down registration information sent in push * quotes * Use async version of load_platform * Add warning for duplicate device names * Switch to async_get_service * add mobile_app.notify test * Update tests/components/mobile_app/test_notify.py * Update tests/components/mobile_app/test_notify.py
* Shutdown ZHAGateway on hass stop. * Cleanup ZHA event leftovers.
* Clean up docstrings * Fix long lines * Fix more docstrings * Fix more docstrings * Fix more docstrings
* Move asyncio condition to instance from class, to be able to pass in lopp. * Avoid not needed sife effects in init methods. * Clean up.
* Google Assistant: Add support for binary sensors * Update test
* Add mobile_app notify platform * Requested changes * Fix incorrect param for status code * Move push_registrations to notify platform file * Trim down registration information sent in push * quotes * Use async version of load_platform * Add warning for duplicate device names * Switch to async_get_service * add mobile_app.notify test * Update tests/components/mobile_app/test_notify.py * Update tests/components/mobile_app/test_notify.py
* Add Hass.io deCONZ discovery flow * add bridge ID * fix attribute * fix strings * Address comments * Add test * Add only instance / changed maybe later
* Dont use DATA_ROOM_SETPOINT_OVRD in climate.opentherm_gw as it is unreliable with some thermostats. * Show new target temperature immediately until the backend notices a change * Only update target temp on the gateway if the value differs from the current target_temperature.
* Fix trusted networks auth provider warning message * Update auth.py
* Deal with cover assumed state * Add docs
* Add circleci support * Add buildpack-deps * Install libudev-dev * sudo * always run test * Add test report * no sugar * quite pytest * better junit test result * Add $CODE_COVERAGE env var
* Add libav depends on circleci * tweak circleci config
* Set up Circleci workflow * Update python tag * Add pre-test job to cache the requirements * Upgrade pip itself * Use 3.7 for lint * Parallelize pylint * Tweak run gen_requirements_all * tweak cache key
* Add mypyrc to control typing check, add mypy to circle * Add translation upload circlci job
* Require static-check success first * Update config.yml
…22696) * wip * added new requirements * fixed tests * improved test coverage * added feed entry's title * added manifest * updated codeowners * generated requirements
* Remove expired 17track.net packages from entity registry * Reverse order
* Update Z-Wave to use async_get_integration * Change load method per PR comments * update tests
* Convert service helper to use async_get_integration * Fix tests
* Remove dependencies and requirements * Revert "Remove dependencies and requirements" This reverts commit fe7171b. * Remove dependencies and requirements * Revert "Remove dependencies and requirements" This reverts commit 391355e. * Remove dependencies and requirements * Fix flake8 complaints * Fix more flake8 complaints * Revert non-component removals
* Migrate check-config to use get_integration * ImportError
* Register services in broadlink domain * Add tests for broadlink services * Resolve review comments * One more review fix * Restore auth retry * Drop unused constants * Fix flake8 errors
* Bandage * lint * move everything into __init__.py * fix lint
* Convert translation helper to use async_get_integration * Simplify after comments * Lint * Fix typing * Typo
* Allow aws credential skip validation * Don't validate the auto-created default profile
* Added editable outputs and multiple zones. * Updated requirements_all.txt * Linter fixes. * Post-review changes * Fixed too many lines separation error * Passing satel controller as parameter to entities. * Fixed linter error. * Fixed forgotten requirements update. * Fixed satel_integra version (again!?!) * Fixed manifest.json. * Fixed passing non-serializable controller * Removed unnecessary isinstance check. * Post review changes
* Fix for #22890 * Singleton count
* Create decorator to check service permissions * Typing * Linting * Member comments * Linting * Member comments * Updated import * Owner comments * Linting * Linting * More work * Fixed tests * Removed service helper tests in RainMachine * Linting * Owner comments * Linting * Owner comments Co-Authored-By: bachya <[email protected]>
* Update manifest validator * Update circle * Update text * Typo * fix link to codeowners * Merge CODEOWNERS into hassfest * Annotate errors with fixable * Convert error to warning * Lint * Make abs path * Python 3.5... * Typo * Fix tests
* Add host selection step to discovery * Review feedback * Fix failing test
* add missing async for tplink's async_setup_platform methods thanks to @MartinHjelmare for spotting this, related to #21916 * fix line lengths
* Start with Stiebel Eltron heatpump * STE HP * Add read of operating mode * Add read-write operation mode * Further extract ModBus access * Separation of platform and API * Last changes * Use modbus hub * Update module doc with config * Clean up platform code * Cleanup and update to dev2 of pystiebeleltron * Remove slave configuration * Add translation of states * Make name parameter optional * Consolidate platform * Correct .coveragerc after conflict * Prepare component for sensor platform * Fix issues found in review * Remove custom states and map to existing HA states * Force update, when values are modified * Update CODEOWNERS and requirements_all.txt * Fix .coveragerc file * Exclude stiebel_eltron components in .coveragerc * Break out to module level constant * Rename constant * Removed REQ and DEP constant.
* Create decorator to check service permissions * Ensure OpenUV service has proper user permissions * Reverting strange changes
austinmroczek
pushed a commit
that referenced
this pull request
Feb 23, 2020
* Significantly reduce code in august integration * Activity updates can now be processed by py-august this allows us to eliminate the activity sync code for the door sensors and locks * Lock and door state can now be consumed from the lock detail api which allows us to remove the status call apis and reduce the number of API calls to august * Refactor the testing method for locks (part #1) * Update homeassistant/components/august/binary_sensor.py Co-Authored-By: Paulus Schoutsen <[email protected]> * Switch to asynctest instead of unittest for mock.patch Co-authored-by: Paulus Schoutsen <[email protected]>
austinmroczek
pushed a commit
that referenced
this pull request
May 6, 2022
* Proof of concept * remove notify platform * remove loose test * Add rework from home-assistant#67912 (#1) * Move notify serviceupdater to Mixins * Move tag discovery handler to Mixins * fix tests * Add typing for async_load_platform_helper * Add add entry unload support for notify platform * Simplify discovery updates * Remove not needed extra logic * Cleanup inrelevant or duplicate code * reuse update_device and move to mixins * Remove notify platform * revert changes to notify platform * Rename update class * unify tag entry setup * Use shared code for device_trigger `update_device` * PoC shared dispatcher for device_trigger * Fix bugs * Improve typing - remove async_update * Unload config_entry and tests * Release dispatcher after setup and deduplicate * closures to methods, revert `in` to `=`, updates * Re-add update support for tag platform * Re-add update support for device-trigger platform * Cleanup rediscovery code revert related changes * Undo discovery code shift * Update homeassistant/components/mqtt/mixins.py Co-authored-by: Erik Montnemery <[email protected]> * Update homeassistant/components/mqtt/device_trigger.py Co-authored-by: Erik Montnemery <[email protected]> * Update homeassistant/components/mqtt/mixins.py Co-authored-by: Erik Montnemery <[email protected]> * revert doc string changes * move conditions * typing and check config_entry_id * Update homeassistant/components/mqtt/mixins.py Co-authored-by: Erik Montnemery <[email protected]> * cleanup not used attribute * Remove entry_unload code and tests * update comment * add second comment Co-authored-by: Erik Montnemery <[email protected]>
austinmroczek
pushed a commit
that referenced
this pull request
Jul 21, 2024
* Initial import 0.0.2 * Fixes to URL, and removed commits * Initial import 0.0.2 * Fixes to URL, and removed commits * Added first test for iotty * First release * Reviewers request #1 - Removed clutter - Added support for new naming convention for IottySmartSwitch entity * Removed commmented code * Some modifications * Modified REST EP for iotty CloudApi * Initial import 0.0.2 * Fixes to URL, and removed commits * Added first test for iotty * First release * Rebased and resolved conflicts * Reviewers request #1 - Removed clutter - Added support for new naming convention for IottySmartSwitch entity * Removed commmented code * Some modifications * Modified REST EP for iotty CloudApi * Removed empty entries in manifest.json * Added test_config_flow * Fix as requested by @edenhaus * Added test_init * Removed comments, added one assert * Added TEST_CONFIG_FLOW * Added test for STORE_ENTITY * Increased code coverage * Full coverage for api.py * Added tests for switch component * Converted INFO logs onto DEBUG logs * Removed .gitignore from commits * Modifications to SWITCH.PY * Initial import 0.0.2 * Fixes to URL, and removed commits * Added first test for iotty * First release * Rebased and resolved conflicts * Fixed conflicts * Reviewers request #1 - Removed clutter - Added support for new naming convention for IottySmartSwitch entity * Removed commmented code * Some modifications * Modified REST EP for iotty CloudApi * Removed empty entries in manifest.json * Added test_config_flow * Some modifications * Fix as requested by @edenhaus * Added test_init * Removed comments, added one assert * Added TEST_CONFIG_FLOW * Added test for STORE_ENTITY * Increased code coverage * Full coverage for api.py * Added tests for switch component * Converted INFO logs onto DEBUG logs * Removed .gitignore from commits * Modifications to SWITCH.PY * Fixed tests for SWITCH * First working implementation of Coordinator * Increased code coverage * Full code coverage * Missing a line in testing * Update homeassistant/components/iotty/__init__.py Co-authored-by: Robert Resch <[email protected]> * Update homeassistant/components/iotty/__init__.py Co-authored-by: Robert Resch <[email protected]> * Modified coordinator as per request by edenhaus * use coordinator entities for switches * move platforms to constants * fix whitespace with ruff-format * correct iotty entry in application_credentials list * minor style improvements * refactor function name * handle new and deleted devices * improve code for adding devices after first initialization * use typed config entry instead of adding known devices to hass.data * improve iotty entity removal * test listeners update cycle * handle iotty as devices and not only as entities * fix test typing for mock config entry * test with fewer mocks for an integration test style opposed to the previous unit test style * remove useless tests and add more integration style tests * check if device_to_remove is None * integration style tests for turning switches on and off * remove redundant coordinator tests * check device status after issuing command in tests * remove unused fixtures * add strict typing for iotty * additional asserts and named snapshots in tests * fix mypy issues after enabling strict typing * upgrade iottycloud version to 0.1.3 * move coordinator to runtime_data * remove entity name * fix typing issues * coding style fixes * improve tests coding style and assertion targets * test edge cases when apis are not working * improve tests comments and assertions --------- Co-authored-by: Robert Resch <[email protected]> Co-authored-by: Shapour Nemati <[email protected]> Co-authored-by: Erik Montnemery <[email protected]> Co-authored-by: shapournemati-iotty <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Breaking Change:
Description:
Related issue (if applicable): fixes #
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for
configuration.yaml
(if applicable):Checklist:
tox
. Your PR cannot be merged unless tests passIf user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
REQUIREMENTS
variable (example).requirements_all.txt
by runningscript/gen_requirements_all.py
..coveragerc
.If the code does not interact with devices: