-
Notifications
You must be signed in to change notification settings - Fork 45
article: MCUboot Getting Start Guide for ESP32 #618
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
base: main
Are you sure you want to change the base?
Conversation
fe53127 to
246809d
Compare
FBEZ
left a comment
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.
Thank you @almir-okato for this article!
I’ve left a few comments for you to review. I also noticed that MCUBoot is in bold every time. I suggest keeping it plain or using a monospaced/verbatim style if needed.
content/blog/2025/12/mcuboot-getting-started/assets/mcuboot-process-overview1.webp
Outdated
Show resolved
Hide resolved
60c251f to
58884b5
Compare
Thanks @FBEZ Regarding the bold style for MCUboot and some other names, I've tried to keep the directions suggested on the review of my previous article in the Developer Portal. |
FBEZ
left a comment
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.
@almir-okato thank you@ I’ve just added a final comment regarding the annotation block.
I still see MCUBoot in bold throughout. Would you like to keep it that way? I find it a bit eye straining.
58884b5 to
f1b3cc5
Compare
I think you are right, I reduced the amount of names in bold, tried to keep only for first occurrences or some highlight notes. Thanks! |
@almir-okato Francesco is right about the formatting of MCUboot and other software names. To be honest, I am not sure why we ended up with all terms in bold in your previous article. I checked all my comments and only found this edit related: #363 (comment) However, I only proposed to make these words bold, because these are the summary words in bullet points, not because all software titles should be in bold. Sorry if I didn't make it clear! |
f-hollow
left a comment
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.
@almir-okato Thank you for contributing this article!
I pointed a out a few nitpicks and shared some ideas on further content polishing.
f1b3cc5 to
3d99b20
Compare
I think I misunderstood that at the time, sorry! I'll try to make it right for the next ones. |
3d99b20 to
e3250f7
Compare
|
@almir-okato Please rebase on main to fix preview deployment. Otherwise, LGTM! Regarding the publishing date, with Christmas and New Year approaching, let’s schedule the publication for, say, 9 January? Releasing it once everyone is back at work will give this article more visibility and focus. If you agree, please update the date. |
Article translated and updated from my original from Embarcados portal. Signed-off-by: Almir Okato <[email protected]>
e3250f7 to
54cebca
Compare
|
🎉 A preview for this PR is available at: https://preview-developer.espressif.com/pr618/ |
|
The last touch: Can you please move the article to To my surprise, I found that this edge case is not verified in our CI. Created a fix in #626 |
Description
Article translated and updated from my original from Embarcados portal.
Related
Testing
Built and visualized using local hugo server.
Checklist
Before submitting a Pull Request, please ensure the following: