Skip to content
This repository has been archived by the owner on Jan 22, 2022. It is now read-only.

Dependency cleanup and API updates #24

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jsoverson
Copy link
Member

  • Removed unused dependencies
  • made WapcHost Send + Sync
  • made WasiParams clone-able

@jsoverson
Copy link
Member Author

cc @autodidaddict: does anything in wasmcloud depend on this crate anymore?

@autodidaddict
Copy link
Collaborator

wasmCloud does not use this crate.

@jsoverson
Copy link
Member Author

Pausing on this for now because making the wasm3-provider Send is not as straightforward as I'd expected. @flavio do you make use of wasm3? @pkedy do you know any consumers?

Its usage dropped off substantially after wasmcloud moved from rust. Since the wasm3-rs repo cautions that it's a WIP, that it may be unsound, and also calls out for a new maintainer, it may be prudent to deprecate wasm3-provider altogether until the status changes.

@flavio
Copy link
Member

flavio commented Oct 27, 2021

@jsoverson kubewarden doesn't make use of this crate, thanks for asking

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants