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

Promote wasmtime-wasi::preview2 to root of crate #7933

Merged
merged 19 commits into from
Feb 21, 2024
Merged

Conversation

pchickey
Copy link
Contributor

Following #7881, wasi-common's re-exports from wasmtime-wasi will be marked as deprecated for the 18.0 release series. This PR lands the change into main to delete those entirely, and promote wasmtime_wasi::preview2 up to wasmtime_wasi's root.

The preview2 feature has been removed, and preview1-on-preview2 has been renamed preview1.

@pchickey pchickey requested review from a team as code owners February 14, 2024 00:44
@pchickey pchickey requested review from alexcrichton and removed request for a team February 14, 2024 00:44
@pchickey pchickey changed the title promote wasmtime-wasi::preview2 to root of crate Promote wasmtime-wasi::preview2 to root of crate Feb 14, 2024
@github-actions github-actions bot added the wasi Issues pertaining to WASI label Feb 14, 2024
@pchickey pchickey added this pull request to the merge queue Feb 14, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 14, 2024
@pchickey pchickey added this pull request to the merge queue Feb 15, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 15, 2024
i guess we didnt build the c-api in CI with deprecation warnings denied

prtest:full
@github-actions github-actions bot added the wasmtime:c-api Issues pertaining to the C API. label Feb 15, 2024
Copy link

Subscribe to Label Action

cc @peterhuene

This issue or pull request has been labeled: "wasi", "wasmtime:c-api"

Thus the following users have been cc'd because of the following labels:

  • peterhuene: wasmtime:c-api

To subscribe or unsubscribe from this label, edit the .github/subscribe-to-label.json configuration file.

Learn more.

@pchickey pchickey added this pull request to the merge queue Feb 21, 2024
Merged via the queue into main with commit a56bd5e Feb 21, 2024
45 checks passed
@pchickey pchickey deleted the pch/promote_preview2 branch February 21, 2024 00:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wasi Issues pertaining to WASI wasmtime:c-api Issues pertaining to the C API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants