Previous Lecture Lecture 09 Next Lecture

Lecture 09, Tue 01/25

Tue Discussion: Work on team01, H05, H06, H07

Tonight:

If your team is finished with team01

… or if you personally have nothing that you can contribute right now, here are three things to work on:

Working on Homework in Pairs

Note that H05, H06 and H07 are all set up to allow pair submission.

An important point about code reviews

When you do an approving code review, there is a right way and a wrong way.

The wrong way is to simply make comments like this:

image

This doesn’t trigger GitHub to mark the Pull Request as approved. To trigger that, here’s what you need to do:

  1. Click where it says “files changed” image

  2. Click where it says “Review Changes”

    image

  3. Click the Approve radio button enter LGTM and then click submit review.

    image

Of course, you are also free to either “just comment” or “request changes” if you feel that’s appropriate. But if your intent is to approve, this is the way to go about it.