git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: Martin <git@mfriebe.de>
To: git@vger.kernel.org
Subject: Re: PATCH: improve git switch documentation
Date: Sat, 17 Jul 2021 09:02:32 +0200	[thread overview]
Message-ID: <db554ab1-11b3-d7e9-6b64-799bc79cb622@mfriebe.de> (raw)
In-Reply-To: <60f1d650e2667_330208e@natae.notmuch>

On 16/07/2021 20:56, Felipe Contreras wrote:
> Martin wrote:
>> On 13/07/2021 18:02, Felipe Contreras wrote:
>>> Martin wrote
>>>> You and I will make the connection between "something happens to the
>>>> branch" and "something happens to the commits".
>>>> A lot of people with less experience, who a busy looking through lots of
>>>> stuff to solve their problem, they will not make that connection in that
>>>> particular moment.
>>>> Heck, I've seen highly educated people missing far more obvious things
>>>> like that.
>>>
>>> Once again I'm not talking about what they could miss, I'm talking about
>>> what they are thinking the command will do.
>>
>> Well they think it creates a new branch with the given name. And that is
>> *all* they think.
> 
> No. You are avoiding the question.

I did not avoid it. I answered it, as I understood it. Seems your 
question was not very clear.

> 
> -c creates a new branch. Obviously -C creates a new branch too.
> 
> Once again, *why* would they pick -C over -c? What do they think it will
> do differently?
> 

They think: it makes go away that error message. They can use that 
branchname.

What they do not think is: If I take away the old branch name, what 
happens to the commits in it?

I know, you firmly believe everyone must surely make that conclusion.
But that fails several times..
1) It assumes everyone has enough knowledge to make that conclusion.
While I agree: "they should", I acknowledge they might not.
But, ok. lets say: "there fault". And we don't give a sh*t if others get 
into problem, because they did not read lots of pages and memorized 
every detail...e

2) It assumes the can.
I.e. they have the experience and skill to make the connection. Ok, 
probably 99% can do.

3) It assumes they do (the attempt to make a connection)
And this is my point. Many people will not attempt to think ahead.


People at that moment think about the branch, and the branch only. Many 
will not an all think about commits.

And why would they. In git there are plenty of situations where you can 
delete a branch, without loosing anything else (i.e. without loosing 
commits), because there is an upstream or another local branch.
Until they day that you pick a branch where there is no safety net.



> 
> Bothering 99.99% of users with a useless warning just because one (who
> is not the sharpest pencil in the box) might make a mistake is just not
> wise.
> 
Well, I see you did a survey over a representative group of randomly 
picked people?

Well, yes I cannot tell you any final number. But from what I observed 
from those people that I know, there a quite a few how mistook that 
documentation.
Many (almost most) of those where lucky, in that they had yet only done 
it, when indeed it was safe. But upon question they were surprised that 
it could have gone another way.

Yes that is not representative. But even if I say that in real live the 
quota of such misunderstanding is at only 10% of what I saw, that would 
be a considerable total.


> 
> That being said, we don't have to agree. And we don't have to
> continuously discuss forever. At some point you need to send a new
> version of your patch, and I think that point is long past due.

Yes but part of this has been educative.

(and some of it a bit of fun too)

And I said I will.
But right now, I have things in my live, that prevent me from doing so 
immediately.
They should prevent me from spending time on those mails too, but I 
can't always withstand - so some shortened nights ahead.

I will look at sending a patch, when I have good time to do so without 
being in any rush.


  reply	other threads:[~2021-07-17  7:02 UTC|newest]

Thread overview: 103+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-29 15:28 PATCH: improve git switch documentation Martin
2021-06-29 16:35 ` Junio C Hamano
2021-06-29 19:04   ` Martin
2021-06-29 22:39     ` Junio C Hamano
2021-06-30  8:50       ` Martin
2021-06-30 22:59         ` Junio C Hamano
2021-07-01 10:06           ` Martin
2021-07-01 11:27             ` Sergey Organov
2021-07-07 18:34               ` Felipe Contreras
2021-07-07 20:46                 ` Sergey Organov
2021-07-07 21:25                   ` What actually is a branch? Felipe Contreras
2021-07-07 22:07                     ` Sergey Organov
2021-07-07 22:35                       ` Martin
2021-07-08  3:39                         ` Felipe Contreras
2021-07-08 10:15                           ` Martin
2021-07-08 17:33                             ` Felipe Contreras
2021-07-08 19:21                               ` Martin
2021-07-08 20:37                                 ` Felipe Contreras
2021-07-08 23:11                                   ` Martin
2021-07-09  0:45                                     ` Felipe Contreras
2021-07-09 13:24                                       ` Martin
2021-07-09 15:08                                         ` Felipe Contreras
2021-07-09 15:23                                           ` switch requires --detach [[Re: What actually is a branch]] Martin
2021-07-09 16:21                                             ` Felipe Contreras
2021-07-09 16:38                                               ` Randall S. Becker
2021-07-09 17:10                                                 ` Felipe Contreras
2021-07-09 16:54                                               ` Martin
2021-07-10 10:08                                             ` Sergey Organov
2021-07-10 19:18                                               ` Felipe Contreras
2021-07-09 14:29                                       ` PATCH: improve git switch documentation Martin
2021-07-09 16:10                                         ` Felipe Contreras
2021-07-09 16:51                                           ` Martin
2021-07-09 17:41                                             ` Felipe Contreras
2021-07-09 18:23                                               ` Martin
2021-07-10 19:45                                                 ` Felipe Contreras
2021-07-10 20:07                                                   ` Martin
2021-07-10 20:49                                                     ` Felipe Contreras
2021-07-10 22:13                                                       ` Martin
2021-07-10 23:35                                                         ` Felipe Contreras
2021-07-11  9:10                                                           ` Martin
2021-07-11  9:30                                                             ` Sergey Organov
2021-07-12 16:28                                                             ` Felipe Contreras
2021-07-12 16:33                                                               ` Martin
2021-07-12 16:58                                                                 ` Felipe Contreras
2021-07-12 17:52                                                                   ` Martin
2021-07-12 19:08                                                                     ` Felipe Contreras
     [not found]                                                                       ` <3a84e4c9-4e48-1cbe-4fe6-150ff56c8508@mfriebe.de>
     [not found]                                                                         ` <60ecbe577a086_a6b702082@natae.notmuch>
2021-07-13 10:42                                                                           ` Martin
2021-07-13 16:02                                                                             ` Felipe Contreras
2021-07-16 18:12                                                                               ` Martin
2021-07-16 18:31                                                                               ` Martin
2021-07-16 18:56                                                                                 ` Felipe Contreras
2021-07-17  7:02                                                                                   ` Martin [this message]
     [not found]                                                                                   ` <1997ca3b-117a-e19a-0dee-7342a2f1a0e7@mfriebe.de>
     [not found]                                                                                     ` <60f1f4c3dd8b1_14cb208c1@natae.notmuch>
     [not found]                                                                                       ` <fedbfe1f-9e6d-f46f-ca41-e176a30e938c@mfriebe.de>
     [not found]                                                                                         ` <60f22aaa6a4f1_1f602081b@natae.notmuch>
2021-07-17 10:07                                                                                           ` Martin
     [not found]                                                                                             ` <60f33f8a7c39b_507220823@natae.notmuch>
2021-07-17 21:23                                                                                               ` Martin
2021-07-19 17:51                                                                                                 ` Felipe Contreras
2021-07-11  7:57                                                         ` Sergey Organov
2021-07-11  9:27                                                           ` Martin
2021-07-11  9:37                                                             ` Sergey Organov
2021-07-11 10:24                                                               ` Martin
2021-07-12 16:34                                                             ` Felipe Contreras
2021-07-10 22:13                                                       ` Naming the --forec option [[Re: PATCH: improve git switch documentation]] Martin
2021-07-10 23:18                                                         ` Felipe Contreras
2021-07-11  0:39                                                           ` Martin
2021-07-12 16:15                                                             ` Felipe Contreras
2021-07-10 10:24                                             ` PATCH: improve git switch documentation Sergey Organov
2021-07-10 10:37                                               ` Bagas Sanjaya
2021-07-10 11:05                                               ` Martin
2021-07-10 16:32                                                 ` Sergey Organov
2021-07-10 20:12                                                   ` Felipe Contreras
2021-07-11  9:04                                                     ` Sergey Organov
2021-07-11 10:05                                                       ` Martin
2021-07-11 12:23                                                         ` Sergey Organov
2021-07-11 13:39                                                           ` Martin
2021-07-11 14:49                                                             ` Sergey Organov
2021-07-11 16:51                                                             ` Sergey Organov
2021-07-12 10:31                                                               ` Kerry, Richard
2021-07-12 11:11                                                                 ` Sergey Organov
2021-07-12 16:55                                                                   ` Felipe Contreras
2021-07-12 16:24                                                       ` Felipe Contreras
2021-07-12 16:39                                                         ` Martin
2021-07-12 17:09                                                           ` Felipe Contreras
2021-07-12 22:58                                                         ` Sergey Organov
2021-07-12 23:36                                                           ` Felipe Contreras
2021-07-13 11:20                                                           ` Martin
2021-07-10 20:00                                                 ` Felipe Contreras
2021-07-10 19:51                                               ` Felipe Contreras
2021-07-11  9:52                                                 ` Sergey Organov
2021-07-12 16:44                                                   ` Felipe Contreras
2021-07-13 10:57                                                     ` Sergey Organov
2021-07-13 16:10                                                       ` Felipe Contreras
2021-07-14 19:14                                                         ` Sergey Organov
2021-07-14 19:51                                                           ` Felipe Contreras
2021-07-14 20:42                                                             ` Sergey Organov
2021-07-08  3:12                       ` What actually is a branch? Felipe Contreras
2021-07-08 11:16                         ` Sergey Organov
2021-07-08 18:05                           ` Felipe Contreras
2021-07-01 14:58             ` PATCH: improve git switch documentation Junio C Hamano
2021-07-01 17:29               ` Martin
2021-07-01 17:46                 ` Sergey Organov
2021-07-07 18:54                 ` Felipe Contreras
2021-07-07 18:47               ` Felipe Contreras
2021-07-07 18:14             ` Felipe Contreras
2021-07-01  0:06         ` Matt Rogers

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: http://vger.kernel.org/majordomo-info.html

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=db554ab1-11b3-d7e9-6b64-799bc79cb622@mfriebe.de \
    --to=git@mfriebe.de \
    --cc=git@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).