-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Bump zwave-js to 9.0.2 and zwave-js-server to 1.16.0 #2424
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.
I wonder if we should delay releasing this a couple of days after the core release to not potentially confuse users about what could be the reason for any issues that come up with their devices?
we can, but may want to check in with @frenck as well to see when he plans to release zwavejs2mqtt, there's no point delaying if the other one gets released anyway. I've messaged him on Discord and will report back here |
🤷 I'm happy to adopt any schedule |
EDIT: ignore what I said there's an inclusion bug that needs to be fixed in v9. We should probably add the fix here before we merge. Doesn't seem like there are any HA related problems though! |
this is ready to be merged. |
can someone restart CI? |
Another job failed this time. |
|
all good now! |
This PR isn't working correctly yet which is why it's still in draft. Details after changelog.This has been resolved. According to this issue we need to build @serialport/bindings-cpp from source: serialport/node-serialport#2438Changelog: