Skip to content

Docs fixes#1143

Merged
drager merged 1 commit intodrager:masterfrom
helixbass:book-fixes
Jan 28, 2023
Merged

Docs fixes#1143
drager merged 1 commit intodrager:masterfrom
helixbass:book-fixes

Conversation

@helixbass
Copy link
Contributor

This PR just includes a couple small fixes (typos/grammar/etc) that I noticed as I was reading the wasm-pack docs

## Scope

The init command also accepts an optional `--scope` argument. This will scope
The `build` command also accepts an optional `--scope` argument. This will scope
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn't test that you can actually pass --scope to wasm-pack build but this would seem to be the correct change here (to be symmetrical with other stuff on this page and to match the example command below)

## 🏃‍♀️ Up and Running

1. fork and clone this repository
1. fork and clone the `rustwasm/wasm-pack` repository
Copy link
Contributor Author

Choose a reason for hiding this comment

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

When reading at https://rustwasm.github.io/docs/wasm-pack/ it wasn't self-evident what "this repository" was

Copy link
Owner

@drager drager left a comment

Choose a reason for hiding this comment

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

Thank you!

@drager drager merged commit 5a59df0 into drager:master Jan 28, 2023
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.

2 participants