Re: [PATCH 10/10] builtin/fetch: check for submodule updates in any ref update
- From
- Jonathan Tan <jonathantanmy@google.com>
- Date
- Oct 26, 2018, 20:42 UTC
- Message-ID
- <20181026204249.132969-1-jonathantanmy@google.com>
- In-Reply-To
- <20181025233231.102245-11-sbeller@google.com>
Show 6 quoted lines
> Gerrit, the code review tool, has a different workflow than our mailing > list based approach. Usually users upload changes to a Gerrit server and > continuous integration and testing happens by bots. Sometimes however a > user wants to checkout a change locally and look at it locally. For this > use case, Gerrit offers a command line snippet to copy and paste to your > terminal, which looks like
As I said in my review of the previous patch, I think this should be squashed into the previous patch.
Also, remember to add the version when formatting the e-mail ("[PATCH v? ??/??]").