Skip to content

Conversation

@almir-okato
Copy link
Contributor

@almir-okato almir-okato commented Dec 13, 2025

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:

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

@almir-okato almir-okato force-pushed the mcuboot_getting_started branch from fe53127 to 246809d Compare December 15, 2025 14:05
@pedrominatel pedrominatel requested a review from FBEZ December 16, 2025 13:21
@pedrominatel pedrominatel added the needs review Needs someone to be assigned to review label Dec 16, 2025
Copy link
Collaborator

@FBEZ FBEZ 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 @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.

@FBEZ FBEZ added Awaiting Response Awaiting a response from the author and removed needs review Needs someone to be assigned to review labels Dec 17, 2025
@almir-okato almir-okato force-pushed the mcuboot_getting_started branch 2 times, most recently from 60c251f to 58884b5 Compare December 17, 2025 15:40
@almir-okato
Copy link
Contributor Author

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.

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.
Should I change for MCUboot and the other terms?

@almir-okato almir-okato requested a review from FBEZ December 17, 2025 18:13
Copy link
Collaborator

@FBEZ FBEZ left a 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.

@almir-okato almir-okato force-pushed the mcuboot_getting_started branch from 58884b5 to f1b3cc5 Compare December 18, 2025 16:30
@almir-okato
Copy link
Contributor Author

@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.

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 almir-okato requested a review from FBEZ December 18, 2025 16:36
@f-hollow
Copy link
Collaborator

f-hollow commented Dec 22, 2025

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.

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. Should I change for MCUboot and the other terms?

@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!

Copy link
Collaborator

@f-hollow f-hollow left a 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.

@almir-okato almir-okato force-pushed the mcuboot_getting_started branch from f1b3cc5 to 3d99b20 Compare December 24, 2025 18:32
@almir-okato
Copy link
Contributor Author

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.

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. Should I change for MCUboot and the other terms?

@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!

I think I misunderstood that at the time, sorry! I'll try to make it right for the next ones.

@almir-okato almir-okato force-pushed the mcuboot_getting_started branch from 3d99b20 to e3250f7 Compare December 24, 2025 18:41
@almir-okato almir-okato requested a review from f-hollow December 24, 2025 18:46
@almir-okato
Copy link
Contributor Author

@FBEZ @f-hollow thanks for your review! I've applied your suggestions.

@f-hollow
Copy link
Collaborator

f-hollow commented Dec 25, 2025

@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]>
@almir-okato almir-okato force-pushed the mcuboot_getting_started branch from e3250f7 to 54cebca Compare December 26, 2025 23:12
@github-actions
Copy link

🎉 A preview for this PR is available at: https://preview-developer.espressif.com/pr618/

@f-hollow f-hollow mentioned this pull request Dec 29, 2025
6 tasks
@f-hollow
Copy link
Collaborator

f-hollow commented Dec 29, 2025

@almir-okato

The last touch: Can you please move the article to 2026/01 folder so that the article date matches the year-month folder?

To my surprise, I found that this edge case is not verified in our CI. Created a fix in #626

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Awaiting Response Awaiting a response from the author

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants