githubtraining / githubtraining/training-manual

Class notes

Open
#172 10 comments 0 reactions 0 assignees View on GitHub
Dominant language
Shell
Stars
340
Forks
375
PR merge metrics
No merged PRs in 30d

Description

Note - I'll come back later and create actual issues out of this. This is notes while I'm helping @MBLACK88 teach a class today 🎆

- [ ] Checking local git config seems oddly placed. Why don't we verify this in the getting started part?
- [ ] on 09_merging_pull_requests, there's the icon for merge, but why not use the button for new branch too?
- [ ] Going over the commit structure might be more helpful in the beginning... 😕
- [ ] Inside of reverting commits, might be helpful to do cherry pick right after. We started doing the revert from game_instructions and fixed inde.html to index.html. Then it was a nice moment to demo cherry pick from here to gh-pages. This was kind of by accident, but the opportunity seemed to flow better.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.