Skip to content

Update getting-started.md, Power switch. #1964

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

Closed
wants to merge 2 commits into from
Closed

Conversation

dbuggz
Copy link
Contributor

@dbuggz dbuggz commented May 20, 2024

Re-wording to emphasize that the robot is better turned off while programming.

Sorry if this seems pedantic, but the original wording confused me.

Also, the Arduino Lab has not been introduced to the reader yet, so some reference to what it is may be helpful.

What This PR Changes

  • (Please explain here why you created the pull request and specify what it changes)

Contribution Guidelines

Re-wording to emphasize that the robot is better turned off while programming.

Sorry if this seems pedantic, but the original wording confused me.

Also, the Arduino Lab has not been introduced to the reader yet, so some reference to what it is may be helpful.
@CLAassistant
Copy link

CLAassistant commented May 20, 2024

CLA assistant check
All committers have signed the CLA.

@dbuggz
Copy link
Contributor Author

dbuggz commented May 20, 2024

In Reference to my note that the Terminal had not been introduced to the reader yet, a reference to "MicroPython Firmware on the Nano ESP32" later in the document, but I'm not sure how to do that.

@jhansson-ard
Copy link
Contributor

@dbuggz Thank you for your PR! Please sign the CLA so that we can properly review your submission ⭐

@jhansson-ard jhansson-ard added the community Bugs and fixes suggested by the community label May 21, 2024
jhansson-ard added a commit that referenced this pull request May 29, 2024
Since the PR #1964 was created, the files have changed. I've added the PR changes from there to here.
@jhansson-ard
Copy link
Contributor

@dbuggz your changes was added in another PR (here: #1986) since this had become outdated.

We've now merged it, thank you for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Bugs and fixes suggested by the community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants