-
Notifications
You must be signed in to change notification settings - Fork 133
docs: Provide CN translation for three docs #1341
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
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
kolipakakondal
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.
Thanks for the PR @Shen7436 LGTM.
|
LGTM! |
|
PS: If you want to preview the HTML generated from the RST files, please go to the Summary tab and download the docs artifacts (see here). Ali is working on adding a link to the PR’s Conversation page so that you can preview the HTML files directly here. cc @alirana01 |
leeebo
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.
@Shen7436 The changes LGTM!
0cici
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.
Hi @Shen7436, thanks for the translation! Left some suggestions. PTAL~
bde5649 to
b9a51b7
Compare
b9a51b7 to
96accaf
Compare
This PR:
prerequisites.rst,installation.rst,troubleshooting.rstbased on Espressif Style Guide