Re: [PATCH 2/2] pull: improve default warning
- From
Elijah Newren <newren@gmail.com>
- Date
- Jun 22, 2021, 15:06 UTC
- Message-ID
- <CABPp-BF1noWhiJadHzjJmnGo8hdZj6Fk7XnZ=u6BVVSGfHE7og@mail.gmail.com>
- In-Reply-To
- <CAMMLpeRa3atkZxEtV--YD6-JSf0Bp9xRw9kS5wSWerxpsGrvrw@mail.gmail.com>
On Mon, Jun 21, 2021 at 8:15 PM Alex Henrie <alexhenrie24@gmail.com> wrote:
Show 5 quoted lines
> > 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/ 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."