Conversation
anderseknert
left a comment
There was a problem hiding this comment.
LGTM. I’m surprised this change doesn’t warrant a major version bump in the dependency though.
|
Fair, the annotation module is marked as The changes are not affecting the real public API and we have not released a 1.0 release in this project, but, if you are uncomfortable, we can plan to stabilize the layer. |
|
No, I didn’t mean in this project but in the dependency, which according to the title is bumped to 1.2.0. But if it’s part of an experimental module, that’s fair game I suppose :) |
We changed a bit the code generated by the
WasmModuleInterfaceannotation, just a mechanical update that improves a bit the performance too by caching a table lookup when invoking functions.