From: Jonathan Tan <jonathantanmy@google.com>
Date: Fri, 26 Oct 2018 18:57:47 GMT
Subject: Re: [PATCH 05/10] submodule: store OIDs in changed_submodule_names
Message-ID: <20181026185747.259713-1-jonathantanmy@google.com>
In-Reply-To: <20181025233231.102245-6-sbeller@google.com>

> Reviewed-by: Jonathan Tan <jonathantanmy@google.com>

Probably better not to include such lines, especially since the review
by me is not yet complete.

Having said that, patches 1-5 look good to me. Patches 1-3 are identical
to the previous version, which I have already reviewed. In patch 4,
Stefan made the code change I suggested [1].

In this patch, compared to the previous version which I have already
reviewed [2], the code is unchanged aside from some variable renaming. I
suggested a commit title change which Stefan has done.

[1] https://public-inbox.org/git/20181017212624.196598-1-jonathantanmy@google.com/
[2] https://public-inbox.org/git/20181017214534.199890-1-jonathantanmy@google.com/