git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: PATCH: improve git switch documentation

From
MMartin <git@mfriebe.de>
Date
Jul 17, 2021, 10:07 UTC
Message-ID
<e57f1d19-d574-5ba5-efc1-abb8ab2a8c01@mfriebe.de>
In-Reply-To
<60f22aaa6a4f1_1f602081b@natae.notmuch>
On 17/07/2021 02:56, Felipe Contreras wrote:
Show 10 quoted lines
> Martin wrote:
> It is the Socratic method. If I tell you "the user will think X" you are
> not going to believe me. Therefore I'm asking you what the user will
> think.
> 
>> But no one is taking them by the hand. No one is asking all those
>> questions to them.
>> So they (new users) will not always make that conclusion.
> 
> What conclusion would they reach?
You realize that your question makes no sense?

If the user does not enter the state of concluding, then they will not reach a conclusion at all.

If you want to challenge my statement, that the user does not start concluding, then you could ask me: Why? To which I have to admit, I do not know, I did not ask those who didn't. And frankly it does not matter. Lets assume we knew "why". Then to remedy that cause, some change would be needed. So most likely the doc would need to be changed to have some trigger added, to overcome that reason. In the end, that means more info in the doc. Same as what is already proposed.

Show 5 quoted lines
>>> What do they think it will happen?
>>
>> They do not think about it at all.
> 
> This doesn't make any sense. They used -C instead of -c for a reason.

For the 4th or 5th time (not going to count the exact number of times I have answered the exact same question)

The reason is the branchname was used, and the wanted to use it again. The doc says they can use it again. So that is all they want at that moment.

Why is there an extra option for doing this, good question but to them at that point in time: not relevant. The doc of the option does not say, that there may be any consequences, so that is good enough at that moment.

If you want, you can call them ignorant. But in their defence they may not even know that. They have whatever other issues to solve at that time. They are happy to have found that option, and they really need to return to whatever other stuff they were doing. So they trust the docs, and the docs have no explicit warning.

Frankly all the above, is a very common pattern that lots of people show at some time or another. Whatever the problem, people go for whatever *appears* to be the easy fix. No thought on what will happen after that. In German their is a saying "Nach mir die Sinnflut".

Show 7 quoted lines
>>
>> Because they did not correctly understand what the net protected the from.
> 
> Users should not be executing commands they don't understand. If a user
> doesn't understand what `git delete-this-branch` does, then he shouldn't
> run it.
> 
How can they check they understand it?

See also the example of the person that makes *two* the conclusions. How can they tell there is nothing further that they need to conclude?

> If the documentation says a command overwrites a branch, and the user
> runs the command anyway,

So is that an admission that people may not always come to the conclusion? I.e., what I said: They do not think about that part at all.

 > and the branch is overwritten, git did what the
> user told it to do, and what happened is the responsibility of the user.
Well, that is a matter of philosophical debate.

It does not say "commits may be affected". I think or hope, that we can agree the effect on commits is something, that is to be concluded. The discrepancy we have is, whether it will be concluded by all users ("all" is to mean a high percentage leaving no significant rest).

If we agree on that "conclusion" statement, then the discrepancy we have can be further deducted on whether there is such an "significant rest" amount of users.

I believe there is. You do afaik not believe this.

But if there is (or "if there were") such an significant amount of users, then it would be a valuable addition to add text, that add the result of that conclusion.

So then all the "what would they think..." question do not really matter. It does not matter what they think, if it is not what they are expected to think. If there is a significant amount of people who for any reason whatsoever do not think this, then I believe the documentation should respect the fact, that those people exist (and more than just as an exception).

Show 9 quoted lines
>>> Let me try yet another analogy.
>>>
>>> If an alarm clock has two buttons "snooze" and "off".
>> How exactly is that connected?
>> This is a random story. Not an analogy.
> 
> The fact that you don't see the analogy doesn't mean it isn't one.
> 
> Did the user click "off" for a reason?

"Off" is not called "force snooze". Off does not require to conclude info, as "-C" does.

Lets say, there is a "change timezone" and a "force change timezone" button, and the first one would reject to work, if an alarm is active, the 2nd would work even if an alarm exists. Then that would be an analogy. Because then the user has to figure out, that changing the timezone would change the displayed hour, and as a consequence clear the alarm.

In your example, where is the conclusion the user has to make?
Show 5 quoted lines
>>> Mistook it for what? What did they expect it was going to happen?
>> I have answered that in great detail, at least 3 times in this mail thread.
> 
> I'm sorry, but no, "they'll think nothing and they'll do it for no
> reason" is not an answer.
Well, that is not what I wrote.
Previous: MartinNext: Martin
Message 101 of 103 in “PATCH: improve git switch documentation”
  1. MartinJun 29, 2021
  2. Junio C HamanoJun 29, 2021
  3. MartinJun 29, 2021
  4. Junio C HamanoJun 29, 2021
  5. MartinJun 30, 2021
  6. Junio C HamanoJun 30, 2021
  7. MartinJul 1, 2021
  8. Sergey OrganovJul 1, 2021
  9. Felipe ContrerasJul 7, 2021
  10. Sergey OrganovJul 7, 2021
  11. What actually is a branch?Felipe Contreras, Jul 7, 2021
  12. Sergey OrganovJul 7, 2021
  13. MartinJul 7, 2021
  14. Felipe ContrerasJul 8, 2021
  15. MartinJul 8, 2021
  16. Felipe ContrerasJul 8, 2021
  17. MartinJul 8, 2021
  18. Felipe ContrerasJul 8, 2021
  19. MartinJul 8, 2021
  20. Felipe ContrerasJul 9, 2021
  21. MartinJul 9, 2021
  22. Felipe ContrerasJul 9, 2021
  23. switch requires --detach [[Re: What actually is a branch]]Martin, Jul 9, 2021
  24. Felipe ContrerasJul 9, 2021
  25. Randall S. BeckerJul 9, 2021
  26. Felipe ContrerasJul 9, 2021
  27. MartinJul 9, 2021
  28. Sergey OrganovJul 10, 2021
  29. Felipe ContrerasJul 10, 2021
  30. MartinJul 9, 2021
  31. Felipe ContrerasJul 9, 2021
  32. MartinJul 9, 2021
  33. Felipe ContrerasJul 9, 2021
  34. MartinJul 9, 2021
  35. Felipe ContrerasJul 10, 2021
  36. MartinJul 10, 2021
  37. Felipe ContrerasJul 10, 2021
  38. Naming the --forec option [[Re: PATCH: improve git switch documentation]]Martin, Jul 10, 2021
  39. Felipe ContrerasJul 10, 2021
  40. MartinJul 11, 2021
  41. Felipe ContrerasJul 12, 2021
  42. MartinJul 10, 2021
  43. Felipe ContrerasJul 10, 2021
  44. MartinJul 11, 2021
  45. Sergey OrganovJul 11, 2021
  46. Felipe ContrerasJul 12, 2021
  47. MartinJul 12, 2021
  48. Felipe ContrerasJul 12, 2021
  49. MartinJul 12, 2021
  50. Felipe ContrerasJul 12, 2021
  51. Sergey OrganovJul 11, 2021
  52. MartinJul 11, 2021
  53. Sergey OrganovJul 11, 2021
  54. MartinJul 11, 2021
  55. Felipe ContrerasJul 12, 2021
  56. Sergey OrganovJul 10, 2021
  57. Bagas SanjayaJul 10, 2021
  58. MartinJul 10, 2021
  59. Sergey OrganovJul 10, 2021
  60. Felipe ContrerasJul 10, 2021
  61. Sergey OrganovJul 11, 2021
  62. MartinJul 11, 2021
  63. Sergey OrganovJul 11, 2021
  64. MartinJul 11, 2021
  65. Sergey OrganovJul 11, 2021
  66. Sergey OrganovJul 11, 2021
  67. Kerry, RichardJul 12, 2021
  68. Sergey OrganovJul 12, 2021
  69. Felipe ContrerasJul 12, 2021
  70. Felipe ContrerasJul 12, 2021
  71. MartinJul 12, 2021
  72. Felipe ContrerasJul 12, 2021
  73. Sergey OrganovJul 12, 2021
  74. Felipe ContrerasJul 12, 2021
  75. MartinJul 13, 2021
  76. Felipe ContrerasJul 10, 2021
  77. Felipe ContrerasJul 10, 2021
  78. Sergey OrganovJul 11, 2021
  79. Felipe ContrerasJul 12, 2021
  80. Sergey OrganovJul 13, 2021
  81. Felipe ContrerasJul 13, 2021
  82. Sergey OrganovJul 14, 2021
  83. Felipe ContrerasJul 14, 2021
  84. Sergey OrganovJul 14, 2021
  85. Felipe ContrerasJul 8, 2021
  86. Sergey OrganovJul 8, 2021
  87. Felipe ContrerasJul 8, 2021
  88. Junio C HamanoJul 1, 2021
  89. MartinJul 1, 2021
  90. Sergey OrganovJul 1, 2021
  91. Felipe ContrerasJul 7, 2021
  92. Felipe ContrerasJul 7, 2021
  93. Felipe ContrerasJul 7, 2021
  94. Matt RogersJul 1, 2021
  95. MartinJul 13, 2021
  96. Felipe ContrerasJul 13, 2021
  97. MartinJul 16, 2021
  98. MartinJul 16, 2021
  99. Felipe ContrerasJul 16, 2021
  100. MartinJul 17, 2021
  101. MartinJul 17, 2021
  102. MartinJul 17, 2021
  103. Felipe ContrerasJul 19, 2021

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.