Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename -S common to -S cli. #8166

Merged
merged 3 commits into from
Mar 18, 2024

Conversation

sunfishcode
Copy link
Member

The "common" in -S common came from "wasi-common" which came from the idea of having code in common between Wasmtime, Lucet, and others. It doesn't have a clear meaning for end users, and has a risk of being interpreted as "common" functionality that's generally available everywhere.

This PR renames -S common to -S cli, and documents it as including the WASI CLI APIs, to clarify its purpose -S common is still accepted, with a warning.

@sunfishcode sunfishcode requested review from a team as code owners March 18, 2024 15:00
@sunfishcode sunfishcode requested review from alexcrichton and removed request for a team March 18, 2024 15:00
@github-actions github-actions bot added the wasmtime:docs Issues related to Wasmtime's documentation label Mar 18, 2024
RELEASES.md Outdated Show resolved Hide resolved
src/commands/run.rs Outdated Show resolved Hide resolved
The "common" in `-S common` came from "wasi-common" which came from the
idea of having code in common between Wasmtime, Lucet, and others. It
doesn't have a clear meaning for end users, and has a risk of being
interpreted as "common" functionality that's generally available
everywhere.

This PR renames `-S common` to `-S cli`, and documents it as including
the WASI CLI APIs, to clarify its purpose `-S common` is still accepted,
with a warning.
@sunfishcode sunfishcode added this pull request to the merge queue Mar 18, 2024
Merged via the queue into bytecodealliance:main with commit bcd0119 Mar 18, 2024
19 checks passed
@sunfishcode sunfishcode deleted the sunfishcode/cli branch March 18, 2024 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wasmtime:docs Issues related to Wasmtime's documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants