-
Notifications
You must be signed in to change notification settings - Fork 82
A few tweaks to quick start guide to make it work #54
Conversation
@@ -13,7 +13,7 @@ first Soroban contract. | |||
Start by creating a new Rust library using the `cargo new` command. | |||
|
|||
```sh | |||
cargo new --lib [project-name] | |||
cargo new --lib first-project |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This originally said [project-name]
but later on in the config the project is referenced as "first-project" .
It's not a big deal but it took me a minute to realize I had to find and replace all first-project
instances with the name I came up with for my test project.
If we make this change the whole page can be followed as a tutorial. I think anyone can figure this name can be replaced with whatever they want.
docs/getting-started/quick-start.mdx
Outdated
Install the `wasm32-unknown-unknown` target on your machine: | ||
|
||
```sh | ||
rustup target add wasm32-unknown-unknown |
There was a problem hiding this 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 step in the Setup
page? I think we should make it part of the setup, rather than this step.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
--wasm target/wasm32-unknown-unknown/release/first-project.wasm \ | ||
--id 1 \ | ||
--fn hello \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops. Thank you!
Just went over this guide end to end with a clean rust install, these are the issues I identified along the way – hope this is helpful.