Skip to content

Bump Chicory to 1.2.0#60

Merged
anderseknert merged 1 commit intoStyraOSS:mainfrom
andreaTP:bump-chicory-1.2.0
Mar 31, 2025
Merged

Bump Chicory to 1.2.0#60
anderseknert merged 1 commit intoStyraOSS:mainfrom
andreaTP:bump-chicory-1.2.0

Conversation

@andreaTP
Copy link
Copy Markdown
Collaborator

We changed a bit the code generated by the WasmModuleInterface annotation, just a mechanical update that improves a bit the performance too by caching a table lookup when invoking functions.

@andreaTP andreaTP requested a review from anderseknert March 31, 2025 17:38
Copy link
Copy Markdown
Contributor

@anderseknert anderseknert left a comment

Choose a reason for hiding this comment

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

LGTM. I’m surprised this change doesn’t warrant a major version bump in the dependency though.

@andreaTP
Copy link
Copy Markdown
Collaborator Author

andreaTP commented Mar 31, 2025

Fair, the annotation module is marked as experimental as we are traking progress to provide the best user experience.
We are making it clear that breaking changes might happen, and using it in a real world integration gives us good feedback.

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.
I'll take advantage of any freedom I can fairly take meanwhile :-)

@anderseknert
Copy link
Copy Markdown
Contributor

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 :)

@anderseknert anderseknert merged commit d8a37ae into StyraOSS:main Mar 31, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants