[5.3] note a bug that makes completing the section difficult#55
[5.3] note a bug that makes completing the section difficult#55adamgreig merged 2 commits intorust-embedded:mainfrom
Conversation
This commit highlights rust-embedded#27 in the tutorial directly and gives users a workaround.
| chapter in Rust Book] for more. | ||
|
|
||
| > **NOTE** If `cargo-embed` prints a lot of warnings here don't worry about it. As of now it does | ||
| > **NOTE**: If `cargo-embed` prints a lot of warnings here don't worry about it. As of now it does |
There was a problem hiding this comment.
The convention in this book is to leave the colon : off. I agree that it's a bit janky, but it's everywhere.
There was a problem hiding this comment.
I pattern matched from this file and didn't take a look at the rest, but this file mostly has the with-colon version! Will change, though.
| > Remote debugging using :1337 | ||
| > init::__cortex_m_rt_main () at mdbook/src/05-meet-your-software/examples/init.rs:19 | ||
| > 19 asm::nop(); | ||
| > ``` |
There was a problem hiding this comment.
Not seeing the actual monitor reset halt in this terminal fragment?
|
Addressed the feedback and yeah, I'll keep an eye out :P I might forget, feel free to tag me in and I can PR it back out. If you'd prefer to keep this PR out though, for whatever reason, feel free; I have the info in my brain now haha. |
BartMassey
left a comment
BartMassey
left a comment
There was a problem hiding this comment.
Lol. Just did a grep and realized that there's more variance in the NOTEs that I remembered. This is good, though: thanks for catching the ones in this chapter.
|
(im unable to merge this myself btw!) |
|
Yes, waiting for another rust-embedded person to glance at it and merge it. If it doesn't happen soon, I'll merge it myself, but always nice to get an extra set of eyeballs. |
3 participants
This commit highlights #27 in the tutorial directly and gives users a workaround.