Skip to content
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

Overhaul pt2: Writing your own application, tooling and guidelines #94

Merged
merged 10 commits into from
Jun 6, 2023

Conversation

SergioGasquez
Copy link
Member

  • Updates Writing your own application and tooling chapters
  • Adds style guidelines

f-hollow and others added 2 commits May 31, 2023 15:11
* docs: Add Rust documentation style guide

(cherry picked from commit f7a307a)

* Add missing extension to rust-doc-style-quide

* Update the Terminology chapter

* Add details on using the terms ESP-IDF and esp-idf

* Add more terms

* Add note on not using italics with Espressif product names

* Improve guidelines on where to add link variables

* Add guideine to apply monospace to image and container names

* Add section on admonitions

* Fix markdown formatting issues

---------

Co-authored-by: Kirill Chalov <kirill.chalov@espressif.com>
* feat: ✨ Update chapter order

* feat: ✨ Add tooling note

* feat: ✨ Update pormpts, console outputs and dev container info

* chore: 🔥 Remove hello-world chapter

* chore: 🔥 Remove hello-world chapter

* feat: ✨ Update information to current state of the template

* feat: ✨ Update panic outputs

* chore: 🔥 Remove hello-world chapter

* feat: ✨ Update blinky code

* feat: ✨ Update button code

* feat: ✨ Update interrupt code

* style: 🎨 Remove personal paths

* style: 🎨 Use uppercase

* feat: ✨ Update esp-idf-template chapter

* feat: ✨ Update espflash chapter

* feat: ✨ Update tooling intro

* feat: ✨ Update VsCode chapter

* chore: 🔥 Remove espmonitor

* Update src/tooling/index.md

Co-authored-by: Kirill <105578077+f-hollow@users.noreply.github.com>

* Update src/writing-your-own-application/index.md

Co-authored-by: Kirill <105578077+f-hollow@users.noreply.github.com>

* docs: 🎨 Update links

* docs: 🎨 List possible ways of creating an app

* docs: 📝 Reformat as list of instructions

* docs: 📝 Remove unnecesary logs

* docs: 📝 Add wokwi vs code section

* docs: 🐛 Fix typo

* docs: ⚡️ Update esp-template with new prompts

* docs: 📝 Improve note about next chapters

* docs: 📝 Update undestanding templates chapters

* docs: 📝 Update Wokwi documentation

* docs: 📝 Move note about cargo generate

* docs: 🎨 Format prompt answers

* docs: 🐛 Fix custom runner command

* docs: 📝 Update espflash section

* docs: 📝 Add support tables for simlating and debugging

* docs: 📝 Update applications section

* docs: 🐛 Fix links

* docs: 🐛 Fix links

* docs: 🐛 Fix links

* docs: 📝 Add note about generating project with Cargo

* docs: 📝 LT suggestions

* docs: 📝 Update esp-idf-hal links

* docs: 📝 Use listing and expand how to build/run templates+

* docs: 📝 Add link to main.rs section

* docs: 📝 Remove next section paragraph

* docs: 📝 Be consistent in template projects name

* docs: 📝 Update section name

* docs: Add comma

Co-authored-by: Kirill <105578077+f-hollow@users.noreply.github.com>

* docs: Fix typo

Co-authored-by: Kirill <105578077+f-hollow@users.noreply.github.com>

* docs: Clarify custom chips

Co-authored-by: Kirill <105578077+f-hollow@users.noreply.github.com>

* docs: 📝 Avoid listing all the supported chips

* docs: ⚡️ Update debbuging support table

* docs: 📝 Apply LT suggestions

* docs: 📝 Apply LT suggestions

* docs: ✨ Add no_std book links

---------

Co-authored-by: Kirill <105578077+f-hollow@users.noreply.github.com>
@SergioGasquez SergioGasquez requested a review from jessebraham May 31, 2023 13:37
Copy link
Contributor

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

@SergioGasquez

A couple of nitpicks.

Once the changes are merged, I will read through the chapters more thoroughly to see if there the content structure can/should be improved, etc.

LGTM!

SergioGasquez and others added 7 commits June 2, 2023 11:56
Co-authored-by: Kirill <105578077+f-hollow@users.noreply.github.com>
Co-authored-by: Kirill <105578077+f-hollow@users.noreply.github.com>
Co-authored-by: Kirill <105578077+f-hollow@users.noreply.github.com>
Co-authored-by: Kirill <105578077+f-hollow@users.noreply.github.com>
@SergioGasquez SergioGasquez merged commit f059680 into esp-rs:main Jun 6, 2023
@SergioGasquez SergioGasquez deleted the feat/overhaul branch June 6, 2023 07:13
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