Thread (40 messages) 40 messages, 7 authors, 2021-06-27

Re: [PATCH 2/2] pull: improve default warning

From: Alex Henrie <hidden>
Date: 2021-06-22 21:22:30

On Tue, Jun 22, 2021 at 9:06 AM Elijah Newren [off-list ref] wrote:
On Mon, Jun 21, 2021 at 8:15 PM Alex Henrie [off-list ref] wrote:
quoted
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.
Thanks for the link, and sorry for not having followed this
conversation closely enough to have seen your previous replies. While
we're on the subject, do you have any thoughts on what (if anything)
more should be done before making the switch to aborting instead of
merging with a warning in `git pull`?

-Alex
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help