Skip to content

Add "Project Overview" and "Project Setup" chapters#94

Merged
SergioGasquez merged 5 commits intoesp-rs:feat/overhaulfrom
jessebraham:feature/overhaul-0
Feb 24, 2026
Merged

Add "Project Overview" and "Project Setup" chapters#94
SergioGasquez merged 5 commits intoesp-rs:feat/overhaulfrom
jessebraham:feature/overhaul-0

Conversation

@jessebraham
Copy link
Member

I'm still working on some finishing touches for the "Project Setup" chapter, but expect to have that wrapped up this morning. Just opening a draft here so at least this first (short) chapter can be reviewed in the meantime. Will update here once the next chapter is ready for review.

Copy link
Member

@MabezDev MabezDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm! thanks!

@jessebraham jessebraham marked this pull request as ready for review February 17, 2026 20:33
@jessebraham
Copy link
Member Author

Okay both chapters have been pushed.

Still not entirely happy with this, I keep going back and forth on what should and shouldn't be included. Would appreciate some input here, happy to elaborate on whatever points, just would like some direction on which points need more detail 😁

@jessebraham jessebraham requested a review from MabezDev February 17, 2026 20:34
Copy link
Member

@SergioGasquez SergioGasquez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The content looks good to me! I would just avoid having hardcoded code snippets in the book as it will make it hard to maintain, we can use anchor as we do in the current version of the training.

Copy link
Member

@SergioGasquez SergioGasquez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks

@SergioGasquez SergioGasquez merged commit 290165e into esp-rs:feat/overhaul Feb 24, 2026
6 checks passed
@jessebraham jessebraham deleted the feature/overhaul-0 branch February 24, 2026 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants