Skip to content

Add migrateOptions to nvm restore methods#1245

Merged
MartinHjelmare merged 3 commits into
home-assistant-libs:mainfrom
MindFreeze:migrateOptions
Jun 25, 2025
Merged

Add migrateOptions to nvm restore methods#1245
MartinHjelmare merged 3 commits into
home-assistant-libs:mainfrom
MindFreeze:migrateOptions

Conversation

@MindFreeze

Copy link
Copy Markdown
Collaborator

For zwave-js/backlog#88

Wasn't sure if I should increase the require_schema. The options were added in 42 but the command would work in earlier versions. The options would just be ignored.

@MartinHjelmare MartinHjelmare added the breaking-change Breaking changes label Jun 24, 2025
@MartinHjelmare

Copy link
Copy Markdown
Contributor

I think it's ok to require schema 42 now for it. It's less surprising.

@MartinHjelmare

Copy link
Copy Markdown
Contributor

Tests need to be updated.

Comment thread zwave_js_server/model/controller/__init__.py Outdated
Comment thread zwave_js_server/model/controller/__init__.py Outdated

@MartinHjelmare MartinHjelmare left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@MartinHjelmare
MartinHjelmare merged commit b2e431f into home-assistant-libs:main Jun 25, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking-change Breaking changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants