Stream: git-wasmtime

Topic: wasmtime / PR #13468 Implement WASI initial cwd configura...


view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 03:48):

angelnereira opened PR #13468 from angelnereira:wasi-initial-cwd to bytecodealliance:main:

This implements the missing configuration path for wasi:cli/environment.initial-cwd discussed in #9695.

Summary:

Tests:

view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 03:48):

angelnereira requested pchickey for a review on PR #13468.

view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 03:48):

angelnereira requested wasmtime-wasi-reviewers for a review on PR #13468.

view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 03:48):

angelnereira requested wasmtime-core-reviewers for a review on PR #13468.

view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 07:40):

github-actions[bot] added the label wasi on PR #13468.

view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 15:11):

:thumbs_up: alexcrichton submitted PR review:

Thanks!

view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 15:11):

:speech_balloon: alexcrichton created PR review comment:

There's no need to add these tests, we don't typically test at this micro-level of granularity. The end-to-end tests are fine. (it becomes extremely onerous to refactor if tests are there for every possible behavior and implementation detail of a function)

view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 15:11):

:speech_balloon: alexcrichton created PR review comment:

For this it's best to skip the test here and rely on testing in cli_tests.rs instead. Given that, can you remove this?

view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 15:30):

:memo: angelnereira submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 15:30):

:speech_balloon: angelnereira created PR review comment:

Makes sense, thanks for the guidance. I’ll remove these low-level checks and keep the coverage at the CLI/end-to-end level.

view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 15:34):

:memo: angelnereira submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 15:34):

:speech_balloon: angelnereira created PR review comment:

thanks. I’ll remove these micro-level tests and keep the behavior covered at the end-to-end level.

view this post on Zulip Wasmtime GitHub notifications bot (May 24 2026 at 16:33):

angelnereira updated PR #13468.

view this post on Zulip Wasmtime GitHub notifications bot (May 25 2026 at 14:32):

:thumbs_up: alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (May 25 2026 at 14:32):

alexcrichton added PR #13468 Implement WASI initial cwd configuration to the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (May 25 2026 at 14:57):

:check: alexcrichton merged PR #13468.

view this post on Zulip Wasmtime GitHub notifications bot (May 25 2026 at 14:57):

alexcrichton removed PR #13468 Implement WASI initial cwd configuration from the merge queue.


Last updated: Jun 01 2026 at 09:49 UTC