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: getting started: add probe-rs docs #4314

Merged
merged 2 commits into from
Jan 28, 2025

Conversation

bradjc
Copy link
Contributor

@bradjc bradjc commented Jan 22, 2025

Pull Request Overview

By default the microbit uses probe-rs but we don't mention it or how to install it. This adds the install to the getting started guide.

Testing Strategy

I did install probe-rs with brew tap probe-rs/probe-rs && brew install probe-rs and cargo flash is now a recognized command on my system.

TODO or Help Wanted

I don't know if there is more we should say about probe-rs.

Documentation Updated

  • Updated the relevant files in /docs, or no updates are required.

Formatting

  • Ran make prepush.

By default the microbit uses probe-rs but we don't mention it or how to
install it.
Copy link
Contributor

@brghena brghena left a comment

Choose a reason for hiding this comment

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

Some updates

doc/Getting_Started.md Outdated Show resolved Hide resolved
doc/Getting_Started.md Outdated Show resolved Hide resolved
doc/Getting_Started.md Show resolved Hide resolved
Co-authored-by: Branden Ghena <brghena@berkeley.edu>
@lschuermann lschuermann enabled auto-merge January 27, 2025 16:10
@ppannuto ppannuto dismissed brghena’s stale review January 28, 2025 00:20

stale—all requested changes have been incorporated

@lschuermann lschuermann added this pull request to the merge queue Jan 28, 2025
Merged via the queue into master with commit 33deb91 Jan 28, 2025
21 checks passed
@lschuermann lschuermann deleted the doc-getting-started-probers branch January 28, 2025 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants