-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Update post 2 with beginner friendly cargo tips #1234
Update post 2 with beginner friendly cargo tips #1234
Conversation
As a relatively new person to Rust, I confused the `.cargo/config.toml` with the global cargo config in my home directory (`~/.cargo/config.toml). While this is pretty obviously wrong in hindsight, since I've never used the `[unstable]` options before, I didn't realize my mistake until this next thing that tripped me up. For `cargo install bootimage`, I think it is reasonable to tell the reader to go into a different directory to execute the command, since it might be the case that the reader has never dealt with different targets before and would have no idea that running `cargo install` for the new target that they just made in a json would be wrong (at least this was me). This could be worded differently than the changes I made, but I think that the addition of these could only benefit a confused reader.
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.
Thanks a lot for the PR, these are some useful clarifications! The only thing that I'm not sure about is the blog_os
prefix in the paths. We don't really use this convention anywhere else, so I fear that this might lead to some confusion too. For example, I could imagine that readers create a blog_os subdirectory within their project folder (which might have a different name).
Yeah I definitely agree, maybe the above change would make sense? I don't know if it is too wordy though. |
Maybe something like "the |
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.
Thanks!
As a relatively new person to Rust, I confused the
.cargo/config.toml
with the global cargo config in my home directory (~/.cargo/config.toml). While this is pretty obviously wrong in hindsight, since I've never used the
[unstable]` options before, I didn't realize my mistake until this next thing that tripped me up.For
cargo install bootimage
, I think it is reasonable to tell the reader to go into a different directory to execute the command, since it might be the case that the reader has never dealt with different targets before and would have no idea that runningcargo install
for the new target that they just made in a json would be wrong (at least this was me).This could be worded differently than the changes I made, but I think that the addition of these could only benefit a confused reader.
I'm new to PRs in general so apologies if I messed up something with the branches!