alexcrichton opened PR #2119 from make-cache-optional
to main
:
This commit moves all of the caching support that currently lives in
wasmtime-environ
into awasmtime-cache
crate and makes it optional. The
goal here is to slim down thewasmtime-environ
crate and clearly separate
boundaries where caching is a standalone and optional feature, not intertwined
with other crates.
yurydelendik submitted PR Review.
fitzgen submitted PR Review.
fitzgen created PR Review Comment:
/// enabled.
fitzgen created PR Review Comment:
Make sure to also add
[package.metadata.docs.rs] all-features = true
to the
Cargo.toml
so users actually see these docs.
fitzgen submitted PR Review.
alexcrichton updated PR #2119 from make-cache-optional
to main
:
This commit moves all of the caching support that currently lives in
wasmtime-environ
into awasmtime-cache
crate and makes it optional. The
goal here is to slim down thewasmtime-environ
crate and clearly separate
boundaries where caching is a standalone and optional feature, not intertwined
with other crates.
alexcrichton submitted PR Review.
alexcrichton created PR Review Comment:
I think I'll probably leave this out for now, but only because the
cache
feature is on-by-default. I think for now we want to avoid pulling in thelightbeam
feature which is susceptible to nightly breakage.
alexcrichton updated PR #2119 from make-cache-optional
to main
:
This commit moves all of the caching support that currently lives in
wasmtime-environ
into awasmtime-cache
crate and makes it optional. The
goal here is to slim down thewasmtime-environ
crate and clearly separate
boundaries where caching is a standalone and optional feature, not intertwined
with other crates.
alexcrichton merged PR #2119.
Last updated: Dec 23 2024 at 13:07 UTC