Stream: git-wasmtime

Topic: wasmtime / PR #8353 Fix running wasi-common tests on CI


view this post on Zulip Wasmtime GitHub notifications bot (Apr 12 2024 at 22:10):

alexcrichton requested wasmtime-core-reviewers for a review on PR #8353.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 12 2024 at 22:10):

alexcrichton requested fitzgen for a review on PR #8353.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 12 2024 at 22:10):

alexcrichton opened PR #8353 from alexcrichton:run-tests to bytecodealliance:main:

Turns out we haven't been running wasi-common tests for some time in CI and they've started failing. Force enable the test at all times and then fix the test failures. The test failures here were introduced in #8277 and weren't caught due to the test not running and the fix was to relax the implementation of fd_pread to avoid taking multiple mutable borrows.

<!--
Please make sure you include the following information:

Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.html

Please ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->

view this post on Zulip Wasmtime GitHub notifications bot (Apr 12 2024 at 22:10):

alexcrichton requested wasmtime-default-reviewers for a review on PR #8353.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 12 2024 at 22:56):

elliottt submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 12 2024 at 23:15):

alexcrichton updated PR #8353.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 12 2024 at 23:19):

alexcrichton has enabled auto merge for PR #8353.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 13 2024 at 00:00):

alexcrichton merged PR #8353.


Last updated: Nov 22 2024 at 16:03 UTC