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

First pass on updating the git branch lesson #550

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

Conversation

kaidamasaki
Copy link
Collaborator

Description

A few sentences describing the overall purpose of these textbook changes.

Questions for the Author

  • Have you updated the weekly learning goals in the pedagogy resource?
  • Have you updated the weekly assessment in Schoology?
  • Have you added links to the source (lucidchart, draw.io) for any diagram?
  • Have you updated the READMEs with links to new or moved resources?

Todos

Are any of these items still in progress?

  • Section X still needs some more detailed examples
  • Still need to add "Additional Resources"

Feedback Requested

Please utilize the built-in GitHub Reviewers functionality to assign necessary reviewers. Use this section to further specify portions of the PR that you would like additional or focused feedback on.

Copy link
Contributor

@tildeee tildeee left a comment

Choose a reason for hiding this comment

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

I think that this is good material, but we'll want the PR material on a different document.

I predict that teaching PRs and branches separately would be better to build/reinforce the ideas, and also not conflate the concepts

Copy link
Contributor

@dHelmgren dHelmgren left a comment

Choose a reason for hiding this comment

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

I think this lesson needs to focus on how Branches work, probably with visual reinforcement of what branching means to the flow of code. Also, I'd include a bit on how to recover if you start working on the wrong branch.

@kaidamasaki
Copy link
Collaborator Author

(Force pushed because I realized I fixed links slightly incorrectly.)

@kaidamasaki
Copy link
Collaborator Author

Notes from Devin's git branch lecture:

  • Drew a diagram on the whiteboard about how branches work. (Railroadish with dots for commits.)
  • Explained that if you work on your own branch then you aren't stepping on anyone else's toes. Provides a safe space to work in without causing problems.
  • This is very useful because we want to commit as frequently as possible since this makes error recovery easier.
  • Introduced the concept of branches. (Entirely separate copies of the repo.)
  • Demonstrated stash for fixing starting on the wrong branch.
  • Talked about (merge) conflict resolution.
  • Dinosaur Archeologists? Criminal Detectives?
  • Before you push to master you should make sure to merge master into your own branch.

@kaidamasaki
Copy link
Collaborator Author

* Wording fixes.
* Add `git branch`
* Add `git pull origin master`
@kaidamasaki
Copy link
Collaborator Author

Week 12: intro to PRs

@CheezItMan CheezItMan changed the base branch from master to main July 9, 2020 22:14
@CheezItMan CheezItMan changed the base branch from main to master July 10, 2020 16:24
@CheezItMan CheezItMan changed the base branch from master to main March 4, 2021 20:09
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.

None yet

3 participants