Skip to content

fix(ci): pass --features to cargo via -- separator in wasm-pack#27

Merged
Alex-Wengg merged 1 commit intomainfrom
fix/wasm-pack-features-flag
Apr 27, 2026
Merged

fix(ci): pass --features to cargo via -- separator in wasm-pack#27
Alex-Wengg merged 1 commit intomainfrom
fix/wasm-pack-features-flag

Conversation

@Alex-Wengg
Copy link
Copy Markdown
Member

@Alex-Wengg Alex-Wengg commented Apr 27, 2026

Summary

  • Fixes WASM CI job failure: error: Found argument '--features' which wasn't expected, or isn't valid in this context
  • wasm-pack build does not accept --features directly; cargo arguments must come after the -- separator

Context

Failing job: https://github.com/FluidInference/text-processing-rs/actions/runs/24977493447/job/73132168878

Before:

wasm-pack build --release --target nodejs --features wasm

After:

wasm-pack build --release --target nodejs -- --features wasm

Test plan

  • CI WASM job passes
  • npm run wasm:build:node succeeds locally
  • npm run wasm:build:web succeeds locally
  • npm run wasm:ci succeeds locally

Open in Devin Review

wasm-pack build does not accept --features directly. Cargo arguments
must come after the -- separator. This was breaking the WASM CI job
with: error: Found argument '--features' which wasn't expected.
Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@Alex-Wengg Alex-Wengg merged commit b6217cc into main Apr 27, 2026
5 checks passed
@Alex-Wengg Alex-Wengg mentioned this pull request Apr 27, 2026
4 tasks
Alex-Wengg added a commit that referenced this pull request Apr 27, 2026
- feat: unified NormalizeOptions API + fix #23 compound concat (#24)
- fix: split trailing punctuation in sentence mode (#21) (#25)
- fix: add disable_bare_second flag (#22) + restore TN abbreviation matching (#26)
- fix(ci): pass --features to cargo via -- separator in wasm-pack (#27)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant