home-assistant-ecosystem / home-assistant-cli

:computer: Command-line tool for Home Assistant
Other
446 stars 69 forks source link

migrate from jsonpath-rw to jsonpath-ng #347

Closed onkelbeh closed 4 years ago

onkelbeh commented 4 years ago

should fix https://github.com/home-assistant-ecosystem/home-assistant-cli/issues/340 and fix https://github.com/home-assistant-ecosystem/home-assistant-cli/issues/322

jsonpath-rw is unmaintained and outdated. jsonpath-rw also causes problems if installed in Home Assistant's environment, these Problems do not appear with jsonpath-ng.

Tested it locally. Works for me.

fabaff commented 4 years ago

The migration to jsonpath-ng requires this review request for Fedora to pass. Otherwise we would not be able to update the RPM package.

onkelbeh commented 4 years ago

OK, thanks.

fabaff commented 4 years ago

Fedora review was done.