cfallin opened PR #2805 from release-0.26.0
to main
:
This PR bumps versions of all crates (Wasmtime to 0.26.0, Cranelift to 0.73.0) and adds release notes.
Please let me know if I missed any important PRs or got the descriptions wrong on any of the release-note entries!
peterhuene submitted PR Review.
peterhuene created PR Review Comment:
Perhaps we should just leave the version number off the assertion so it's not a maintenance issue on version bumps?
peterhuene submitted PR Review.
cfallin updated PR #2805 from release-0.26.0
to main
.
cfallin submitted PR Review.
cfallin created PR Review Comment:
Good point -- now it's just
.starts_with("Module was compiled with incompatible Wasmtime version")
. Thanks!
bjorn3 submitted PR Review.
bjorn3 created PR Review Comment:
.starts_with("Module was compiled with an incompatible Wasmtime version")),
cfallin submitted PR Review.
cfallin created PR Review Comment:
This is matching an explicit error string that doesn't have the "an"; are you suggesting changing the wording of the error itself? Perhaps that's best as a separate PR?
bjorn3 submitted PR Review.
bjorn3 created PR Review Comment:
The maintenance issue could be prevented by overwriting the whole version number instead of just the first part I think. Keeping the version number could still be useful.
peterhuene submitted PR Review.
peterhuene created PR Review Comment:
I think this only reads incorrectly when truncated and my opinion is that it doesn't need changing, but we can update the error message in another PR if necessary, I think.
cfallin submitted PR Review.
cfallin created PR Review Comment:
That gets a little more complex, I think, especially when the version string gets longer (0.100.0)... this is a little too in-the-weeds IMHO for a simple "does the serialization check the version" test; any change in the version should suffice and any error message that starts with "Module was compiled with incompatible Wasmtime version" is certainly the right error that we want to see. If @peterhuene wants to make the test fancier in the future then we can certainly do that but I don't think we should hold the release for that.
cfallin merged PR #2805.
Last updated: Dec 23 2024 at 13:07 UTC