Skip to content

wit-bindgen: Add an option to use anyhow::Result instead of wasmtime::Result #12331

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.

Sign up for GitHub

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

Jump to bottom
Merged
fitzgen merged 3 commits into from
Jan 13, 2026
Merged

wit-bindgen: Add an option to use anyhow::Result instead of wasmtime::Result #12331

fitzgen merged 3 commits into from
Jan 13, 2026

Conversation

@fitzgen
Copy link
Member

@fitzgen fitzgen commented Jan 13, 2026

Depends on #12308

@fitzgen fitzgen requested review from a team as code owners January 13, 2026 00:44
@fitzgen fitzgen requested review from alexcrichton and removed request for a team January 13, 2026 00:45
@fitzgen fitzgen force-pushed the wit-bindgen-to-wasmtime-result-option branch from 8217f2f to 6ccf9a9 Compare January 13, 2026 00:47
alexcrichton
alexcrichton approved these changes Jan 13, 2026
View reviewed changes
Copy link
Member

@alexcrichton alexcrichton left a comment

Choose a reason for hiding this comment

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

Can you add this option to the reference here too?

@github-actions github-actions bot added fuzzing Issues related to our fuzzing infrastructure wasmtime:api Related to the API of the `wasmtime` crate itself wizer Issues related to Wizer snapshotting, pre-initialization, and the `wasmtime wizer` subcommand labels Jan 13, 2026
@github-actions
Copy link

github-actions bot commented Jan 13, 2026

Subscribe to Label Action

cc @fitzgen

Details This issue or pull request has been labeled: "fuzzing", "wasmtime:api", "wizer"

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

  • fitzgen: fuzzing, wizer

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

Learn more.

@fitzgen fitzgen enabled auto-merge January 13, 2026 17:25
@fitzgen fitzgen added this pull request to the merge queue Jan 13, 2026
@fitzgen fitzgen removed this pull request from the merge queue due to a manual request Jan 13, 2026
@fitzgen
Copy link
Member Author

fitzgen commented Jan 13, 2026

(Forgot that this depends on #12308, which needs to land first)

@fitzgen fitzgen force-pushed the wit-bindgen-to-wasmtime-result-option branch from 39a036c to 567a7f1 Compare January 13, 2026 18:33
@fitzgen fitzgen enabled auto-merge January 13, 2026 18:33
@fitzgen fitzgen added this pull request to the merge queue Jan 13, 2026
Merged via the queue into bytecodealliance:main with commit fb1827e Jan 13, 2026
45 checks passed
@fitzgen fitzgen deleted the wit-bindgen-to-wasmtime-result-option branch January 13, 2026 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alexcrichton alexcrichton alexcrichton approved these changes

Assignees

No one assigned

Labels

fuzzing Issues related to our fuzzing infrastructure wasmtime:api Related to the API of the `wasmtime` crate itself wizer Issues related to Wizer snapshotting, pre-initialization, and the `wasmtime wizer` subcommand

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@fitzgen @alexcrichton