openhab / openhabian

openHABian - empowering the smart home, for Raspberry Pi and Debian systems
https://community.openhab.org/t/13379
ISC License
818 stars 251 forks source link

[WIP] Fix JS Scripting npm install & Display warning in case of breaking changes #1760

Closed florian-h05 closed 1 year ago

florian-h05 commented 1 year ago

After releasing a new major version of openhab-js, I noticed that "npm install openhab" does not work with new major versions. This fixes this behaviour by using the latest tag.

It checks for breaking change (new major version number) and displays a warning before upgrading, so the user can decide to upgrade later.

Signed-off-by: Florian Hotze florianh_dev@icloud.com

mstormi commented 1 year ago

thanks

florian-h05 commented 1 year ago

@mstormi Please wait with merging, this is actually WIP and not finished.

mstormi commented 1 year ago

upps :) I read WIP the moment I clicked merge

florian-h05 commented 1 year ago

No problem. I will provide a follow-up PR if this one needs any changes.

mstormi commented 1 year ago

Please do. It's merged to main only not yet forwarded to openHAB3 branch.