async-rs / async-rs/futures-timer

Please consider making wasm dependencies target-specific for wasm targets

Open
#62 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
219
Forks
42
PR merge metrics
No merged PRs in 30d

Description

Rather than having a feature to enable to work on wasm, and then erroring out on wasm targets if that feature isn't enabled, please consider making those dependencies target-specific instead (or in addition), so that it never gets pulled in on non-wasm targets.

Contributor guide

No contributing guide indexed for this repository

Research direction

Inspect the Rust dependency and feature declarations, likely in the project manifest, and compare how dependencies are selected for wasm and non-wasm targets. Verify that wasm-only dependencies are not pulled into non-wasm builds and that wasm builds still work without requiring an extra feature.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust, wasm
Domain
build-system
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.