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

fixes to SO-100 readme #600

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

fixes to SO-100 readme #600

wants to merge 2 commits into from

Conversation

philfung
Copy link

What this does

Fixes to the S0-100 instructions

  • Mac: added Mac-specific instructions. the previous instructions were only for Ubuntu
  • readability: increased readability by using numbered lists and sublists (see below for before/after)
  • links: links to youtube videos are to the exact sections of the video instead of to the start

How it was tested

  • looked through README, made sure instructions made sense for both ubuntu and Mac
  • I ran through and made the entire arm with a Mac using the README instructions

Possible reviewers: @jess-moss @Cadene @aliberts

before

Screenshot 2024-12-26 at 6 35 56 PM

after

Screenshot 2024-12-26 at 6 35 15 PM

Philip Fung and others added 2 commits December 26, 2024 18:26
- added Mac-specific instructions in addition to
Ubuntu instructions
- formatting: indexed list and sublist for readability
- links: links to youtube videos are to the exact sections
of the video instead of to the start

test plan:
- looked through README, made sure instructions made sense for both
 ubuntu and Mac
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.

1 participant