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

docs: fix invalid reference to chip revision (IDFGH-10497) #11743

Closed
wants to merge 2 commits into from

Conversation

crespum
Copy link
Contributor

@crespum crespum commented Jun 26, 2023

According to the official errata document for the ESP32-C3, revisions are named 0.1, 0.2 and so on (instead of 1, 2...).

@CLAassistant
Copy link

CLAassistant commented Jun 26, 2023

CLA assistant check
All committers have signed the CLA.

@espressif-bot espressif-bot added the Status: Opened Issue is new label Jun 26, 2023
@github-actions github-actions bot changed the title docs: fix invalid reference to chip revision docs: fix invalid reference to chip revision (IDFGH-10497) Jun 26, 2023
Copy link
Collaborator

@krzychb krzychb left a comment

Choose a reason for hiding this comment

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

Hi @crespum, I left one comment.

Thank you for spotting and correcting this issue!

docs/en/api-guides/jtag-debugging/index.rst Outdated Show resolved Hide resolved
@krzychb
Copy link
Collaborator

krzychb commented Jun 28, 2023

sha=00e903fd3a1036ab40bab5a2884b7d70acb2832f

@krzychb krzychb added the PR-Sync-Merge Pull request sync as merge commit label Jun 28, 2023
@espressif-bot espressif-bot added Status: Selected for Development Issue is selected for development Status: Done Issue is done internally Resolution: NA Issue resolution is unavailable and removed Status: Opened Issue is new Status: Selected for Development Issue is selected for development labels Jun 29, 2023
@krzychb
Copy link
Collaborator

krzychb commented Jul 20, 2023

Merged in 9cab62e

@krzychb krzychb closed this Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR-Sync-Merge Pull request sync as merge commit Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants