Re: [PATCH 2/2] pull: improve default warning
From: Elijah Newren <hidden>
Date: 2021-06-22 15:06:47
On Mon, Jun 21, 2021 at 8:15 PM Alex Henrie [off-list ref] wrote:
My only serious objection to this patch is the instruction to merge if you don't know what to do instead of asking the repository maintainer what to do or reading the Git documentation. I don't have a strong opinion on the rest of the patch.
You're not alone, Alex; I objected to that part as well. (See e.g. https://lore.kernel.org/git/CABPp-BF4rXBOKsn8bG6y3QUEtNVV9K2Pk5NmwrU5818CqhRt_Q@mail.gmail.com/ (local) and various other emails in that thread, ending with "agree to disagree" later). I still object to it as I did then. I'm curious whether it'll just be resubmitted again multiple times, eventually with a cover letter that repeats something along the lines of "these are the non-controversial changes from last-year series which...don't have any reason not to be merged."