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

sdk: Add wasm target for docs #3901

Merged
merged 1 commit into from
Dec 4, 2024
Merged

Conversation

joncinque
Copy link

Problem

As pointed out in this PR comment it's good to specify that docs build for wasm32 where needed.

Also, solana-sdk and solana-program don't configure docs builds with all features enabled.

Summary of changes

In most places, add "wasm32-unknown-unknown" as a build target. For sdk and program, also configure the features to be enabled.

#### Problem

As pointed out in [this PR
comment](anza-xyz#3634 (comment))
it's good to specify that docs build for wasm32 where needed.

Also, solana-sdk and solana-program don't configure docs builds with all
features enabled.

#### Summary of changes

In most places, add "wasm32-unknown-unknown" as a build target. For sdk
and program, also configure the features to be enabled.
@joncinque joncinque added the automerge automerge Merge this Pull Request automatically once CI passes label Dec 4, 2024
@mergify mergify bot merged commit dc51ad7 into anza-xyz:master Dec 4, 2024
53 checks passed
@joncinque joncinque deleted the wasmdocs branch December 4, 2024 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge automerge Merge this Pull Request automatically once CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants