carpentries-incubator / git-novice-branch-pr

Version Control with Git
https://carpentries-incubator.github.io/git-novice-branch-pr
Other
5 stars 17 forks source link

Ideas after teaching workshop #22

Open jmvera255 opened 3 years ago

jmvera255 commented 3 years ago

@sstevens2

After teaching the workshop yesterday, I had the following ideas:

sstevens2 commented 3 years ago

Thanks for the feedback @jmvera255! Some thoughts below.

  • make a mini-workshop version of the lesson that fits within ~3.h hours. Currently the estimated timing of this lesson is 4 hours.

What would you cut in this shorter version? We've kind of left it up to individual instructors to cut at present. When I taught it in May I did manage to get to PRs in 3.5 hrs. Though it would have been nice to spend a little longer with the remotes lesson.

  • I'd like to see the Remotes in GitHub episode come sooner in the workshop which looks like this is the case in the main SWC version but additionally do more work in GitHub in parallel with local Git, for example would it be possible to setup the conflict example to occur when pulling a remote from GitHub or something like that?

I said the same thing in a convo I had with @ChristinaLK after the workshop. It isn't so much that the remotes lesson really comes earlier in the swc git novice lesson but rather that we moved up conflicts so it works with branches instead of doing the conflict in the remote with a collaborator like swc lesson does. Having taught the swc way many times before, it isn't always predicable that both collaborators will edit in the right way to get a conflict (and the added difficulty of partnering people up, esp online, is complicated). I also think doing the conflict in the cloud increases the cognitive load compared to doing it in branches. I do think we should move the remotes lesson up, but I would probably move it up before branches or maybe before gitignore. Moving up the remotes lesson might take a bit of work since then you would want to have them continue to update their repo in the lessons that come after it (which would be good practice pushing and pulling). Would you be interested in working on a PR to make those changes? If not, I'll probably get it it later in spring.

sstevens2 commented 3 years ago

One downside to moving up when we teach github is for workshops where we don't teach github. The way the lesson is split now has you learn all the local stuff without using github at all first.

t4k commented 3 years ago

We've been teaching Carpentry workshops differently over Zoom than we do in person this last year: 1-hour sessions over multiple days to combat Zoom-fatigue.

I just finished 2 one-hour days of "Advanced Git" in which I adapted parts of your customized lesson and they worked really well! Thank you for this work!

A few notes:

Thanks again!

—Tommy (& the Caltech Library Carpentry Team)

ChristinaLK commented 3 years ago

these notes are really useful! Thanks @t4k

sstevens2 commented 3 years ago

Seconding the thanks @t4k ! This is really great feedback to hear! Your timings are also really helpful!