Skip to content

Conversation

@dicej
Copy link
Contributor

@dicej dicej commented Jan 21, 2026

Recently, the above stopped building. This fixes the damage and adds CI coverage.

@dicej dicej requested a review from alexcrichton January 21, 2026 15:30
@dicej dicej requested review from a team as code owners January 21, 2026 15:30
@alexcrichton
Copy link
Member

We do test the p3 feature in CI FWIW so everything runtime-passes, but the hole is that it's tested along with other crates which means that sibling packages might enable features that wasmtime-wasi-http relies on to build. This sort of PR is expected in that sense to be needed every now and then

@dicej
Copy link
Contributor Author

dicej commented Jan 21, 2026

I just realized this PR doesn't do what I said it does anyway. The CI change only enables cargo check coverage.

@dicej dicej force-pushed the fix-wasmtime-wasi-http-p3 branch from 6d4caa6 to 186cdbc Compare January 21, 2026 15:54
@dicej dicej changed the title fix wasmtime-wasi-http p3 tests and run them in CI fix cargo test wasmtime-wasi-http --no-default-features --features p3 Jan 21, 2026
@dicej dicej enabled auto-merge January 21, 2026 16:04
@dicej dicej added this pull request to the merge queue Jan 21, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 21, 2026
webpki-roots = { workspace = true, optional = true }

[dev-dependencies]
wasmtime-wasi-http = { workspace = true, features = ['default-send-request'] }
Copy link
Member

Choose a reason for hiding this comment

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

This is where the error comes from, and you can fix it by replacing workspace = true with path = "." (helps resolve the circular dep cargo-wise)

Recently, the above stopped building.  This fixes the damage and adds CI
coverage.
@dicej dicej force-pushed the fix-wasmtime-wasi-http-p3 branch from 186cdbc to 9ccd77e Compare January 21, 2026 17:54
@dicej dicej enabled auto-merge January 21, 2026 17:54
@dicej dicej added this pull request to the merge queue Jan 21, 2026
Merged via the queue into bytecodealliance:main with commit 4d75577 Jan 21, 2026
45 checks passed
@dicej dicej deleted the fix-wasmtime-wasi-http-p3 branch January 21, 2026 18:30
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.

3 participants