-
-
Notifications
You must be signed in to change notification settings - Fork 605
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
Support for migration from libolm #3978
Conversation
3310a11
to
cf7b115
Compare
489ce34
to
cbac65f
Compare
971471f
to
8c1e897
Compare
This will be faster if we need to prepare the store.
8c1e897
to
7252f7e
Compare
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.
LGTM
const testOlmMachine = makeTestOlmMachine(); | ||
jest.spyOn(OlmMachine, "initialize").mockResolvedValue(testOlmMachine); | ||
jest.spyOn(OlmMachine, "init_from_store").mockResolvedValue(testOlmMachine); |
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.
Surprised to see snake case. A missing jsName in wasm bindings?
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.
yeah :(. Unfortunately fixing it will mean a new release of the bindings. I'll leave it for now and fix it in a future version of the bindings.
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.
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.
Looks sane otherwise
Fixes #3964.
Here's what your changelog entry will look like:
✨ Features