{"thread":{"id":"59698","subject":"[PATCH] doc: merge: improve conflict presentation docs","startedAt":"2023-05-05T13:59:04Z","lastAt":"2023-05-08T15:25:21Z","messageCount":6,"participants":["Adam Johnson via GitGitGadget","Junio C Hamano","Elijah Newren","Felipe Contreras"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"476621","messageId":"pull.1505.git.git.1683295133304.gitgitgadget@gmail.com","threadId":"59698","inReplyTo":null,"subject":"[PATCH] doc: merge: improve conflict presentation docs","fromName":"Adam Johnson via GitGitGadget","fromEmail":"gitgitgadget@gmail.com","sentAt":"2023-05-05T13:58:53Z","receivedAt":"2023-05-05T13:59:04Z","isPatch":true,"sender":{"key":"me@adamj.eu","avatar":"https://avatars.githubusercontent.com/u/857609?v=4"},"body":"From: Adam Johnson \n\nImprovements:\n\n1. Remove the sexist example (\"Barbie... wants to go shopping\")\n2. Show real merge marker contents, rather than e.g. \"yours:sample.txt\".\n3. Swap yours/theirs terms for source/target.\n4. General wordsmithing.\n\nSigned-off-by: Adam Johnson \n---\n doc: merge: improve conflict presentation docs\n \n Improvements:\n \n 1. Remove the sexist example (\"Barbie... wants to go shopping\")\n 2. Show real merge marker contents, rather than e.g.\n \"yours:sample.txt\".\n 3. Swap yours/theirs terms for source/target.\n 4. General wordsmithing.\n\nPublished-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1505%2Fadamchainz%2Faj%2Fmerge-conflict-docs-improvements-v1\nFetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1505/adamchainz/aj/merge-conflict-docs-improvements-v1\nPull-Request: https://github.com/git/git/pull/1505\n\n Documentation/git-merge.txt | 75 ++++++++++++++++++-------------------\n 1 file changed, 37 insertions(+), 38 deletions(-)\n\ndiff --git a/Documentation/git-merge.txt b/Documentation/git-merge.txt\nindex 0aeff572a59..b4523362e48 100644\n--- a/Documentation/git-merge.txt\n+++ b/Documentation/git-merge.txt\n@@ -233,11 +233,11 @@ HOW CONFLICTS ARE PRESENTED\n \n During a merge, the working tree files are updated to reflect the result\n of the merge. Among the changes made to the common ancestor's version,\n-non-overlapping ones (that is, you changed an area of the file while the\n-other side left that area intact, or vice versa) are incorporated in the\n-final result verbatim. When both sides made changes to the same area,\n+non-overlapping ones, where only one side changed an area, are incorporated\n+in the final result as-is. When both sides made changes to the same area,\n however, Git cannot randomly pick one side over the other, and asks you to\n-resolve it by leaving what both sides did to that area.\n+resolve it. Git adds changes from both sides, and optionally the\n+original content from the common ancestor, surrounded by merge markers.\n \n By default, Git uses the same style as the one used by the \"merge\" program\n from the RCS suite to present such a conflicted hunk, like this:\n@@ -245,71 +245,70 @@ from the RCS suite to present such a conflicted hunk, like this:\n ------------\n Here are lines that are either unchanged from the common\n ancestor, or cleanly resolved because only one side changed,\n-or cleanly resolved because both sides changed the same way.\n-<<<<<<< yours:sample.txt\n-Conflict resolution is hard;\n-let's go shopping.\n+or cleanly resolved because both sides changed identically.\n+<<<<<<< HEAD\n+Git makes conflict resolution straightforward.\n =======\n Git makes conflict resolution easy.\n->>>>>>> theirs:sample.txt\n+>>>>>>> main\n And here is another line that is cleanly resolved or unmodified.\n ------------\n \n The area where a pair of conflicting changes happened is marked with markers\n `<<<<<<<`, `=======`, and `>>>>>>>`. The part before the `=======`\n-is typically your side, and the part afterwards is typically their side.\n+is typically from the target (where you’re merging into), and the part\n+afterwards is typically from the source (where you’re merging from).\n+\n+The default format does not show what the original version contained in the\n+conflicting area. You cannot tell how many lines have been deleted and\n+replaced on either side. The only thing you can tell is that the target side\n+says \"straightforward\", while the source side says \"easy\".\n \n-The default format does not show what the original said in the conflicting\n-area. You cannot tell how many lines are deleted and replaced with\n-Barbie's remark on your side. The only thing you can tell is that your\n-side wants to say it is hard and you'd prefer to go shopping, while the\n-other side wants to claim it is easy.\n+You can use an alternative conflict marker style by setting the\n+`merge.conflictStyle` configuration variable to either \"diff3\" or \"zdiff3\".\n+Both of these styles show the original version of the conflicted area, which\n+may help you find a better resolution.\n \n-An alternative style can be used by setting the \"merge.conflictStyle\"\n-configuration variable to either \"diff3\" or \"zdiff3\". In \"diff3\"\n-style, the above conflict may look like this:\n+In the \"diff3\" style, the above conflict looks like this:\n \n ------------\n Here are lines that are either unchanged from the common\n ancestor, or cleanly resolved because only one side changed,\n-<<<<<<< yours:sample.txt\n-or cleanly resolved because both sides changed the same way.\n-Conflict resolution is hard;\n-let's go shopping.\n-||||||| base:sample.txt\n+<<<<<<< HEAD\n or cleanly resolved because both sides changed identically.\n+Git makes conflict resolution straightforward.\n+||||||| 81821ce\n+or cleanly resolved because both sides changed the same way.\n Conflict resolution is hard.\n =======\n-or cleanly resolved because both sides changed the same way.\n+or cleanly resolved because both sides changed identically.\n Git makes conflict resolution easy.\n->>>>>>> theirs:sample.txt\n+>>>>>>> main\n And here is another line that is cleanly resolved or unmodified.\n ------------\n \n-while in \"zdiff3\" style, it may look like this:\n+while in the \"zdiff3\" style, it looks like this:\n \n ------------\n Here are lines that are either unchanged from the common\n ancestor, or cleanly resolved because only one side changed,\n-or cleanly resolved because both sides changed the same way.\n-<<<<<<< yours:sample.txt\n-Conflict resolution is hard;\n-let's go shopping.\n-||||||| base:sample.txt\n or cleanly resolved because both sides changed identically.\n+<<<<<<< HEAD\n+Git makes conflict resolution straightforward.\n+||||||| 81821ce\n+or cleanly resolved because both sides changed the same way.\n Conflict resolution is hard.\n =======\n Git makes conflict resolution easy.\n->>>>>>> theirs:sample.txt\n+>>>>>>> main\n And here is another line that is cleanly resolved or unmodified.\n ------------\n \n-In addition to the `<<<<<<<`, `=======`, and `>>>>>>>` markers, it uses\n-another `|||||||` marker that is followed by the original text. You can\n-tell that the original just stated a fact, and your side simply gave in to\n-that statement and gave up, while the other side tried to have a more\n-positive attitude. You can sometimes come up with a better resolution by\n-viewing the original.\n+The original commit SHA and text are shown after another marker, `|||||||`.\n+This region lets you now see that both sides made the edit from \"the same\n+way\" to \"identically\", as well as editing the following line. The \"diff3\"\n+style keeps all changed lines within the markers, whilst the \"zdiff3\" style\n+moves the commonly edited line before the marker.\n \n \n HOW TO RESOLVE CONFLICTS\n\nbase-commit: 69c786637d7a7fe3b2b8f7d989af095f5f49c3a8\n-- \ngitgitgadget\n"},{"id":"476660","messageId":"xmqqzg6ibe42.fsf@gitster.g","threadId":"59698","inReplyTo":"pull.1505.git.git.1683295133304.gitgitgadget@gmail.com","subject":"Re: [PATCH] doc: merge: improve conflict presentation docs","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2023-05-05T22:35:57Z","receivedAt":"2023-05-05T22:36:02Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"Adam Johnson via GitGitGadget\" writes:\n\n> From: Adam Johnson \n>\n> Improvements:\n>\n> 1. Remove the sexist example (\"Barbie... wants to go shopping\")\n\n\"Barbie goes shopping\" is a pretty common meme. A random internet\nsearch finds many of them, e.g.\n\n https://featuredanimation.com/barbie-memes/\n\nIf it is about only Barbie herself, not about any other random girls,\nwould it still be a \"sexist\" example?\n\n> 2. Show real merge marker contents, rather than e.g. \"yours:sample.txt\".\n\nI am a bit torn about this change. While I can see why we may want\nto show sample output that is bit-for-bit-faithful to reality, these\nexamples are written to teach what each part of the output means,\nand comments like \"yours:sample.txt\" are used instead of the actual\nconflict marker comments to be more helpful for illustrative\npurposes, and this change goes directly against it.\n\n> 3. Swap yours/theirs terms for source/target.\n\nI am not sure if this change is as an improvement, especially in the\ncontext of \"git merge\", which you merge their work into your history\n[*]. Surely, it *is* possible to rephase what I just said to \"you\nmerge source's work into target's history\", but it makes the primary\npoint of doing a merge less clear. But others may have different\nopinions, so please do not take this as the final rejection on this\npoint.\n\nThanks.\n\n\n[Footnote]\n* Conflicts during \"git rebase\" is a different matter. You\n tentatively put your feet in the shoes of your upstream people,\n whom you can call \"them\", and replay \"your\" changes on top of\n \"their\" work, and a conflict you will see during this process is\n what \"they\" will see, i.e. while you are resolving such a\n conflict, you are playing the role of your upstream people, and\n the conflict you see is shown from their point of view. In that\n context, \"your\" vs \"their\" may be suboptimal.\n"},{"id":"476684","messageId":"CABPp-BE50neqaQbE2tTq_=fEM9j_8-cxgj7xKcSfAunoWjsigw@mail.gmail.com","threadId":"59698","inReplyTo":"pull.1505.git.git.1683295133304.gitgitgadget@gmail.com","subject":"Re: [PATCH] doc: merge: improve conflict presentation docs","fromName":"Elijah Newren","fromEmail":"newren@gmail.com","sentAt":"2023-05-06T23:40:09Z","receivedAt":"2023-05-06T23:40:29Z","isPatch":true,"sender":{"key":"newren@gmail.com","avatar":"https://avatars.githubusercontent.com/u/5455730?v=4"},"body":"On Fri, May 5, 2023 at 6:58 AM Adam Johnson via GitGitGadget\n wrote:\n>\n> From: Adam Johnson \n>\n> Improvements:\n>\n> 1. Remove the sexist example (\"Barbie... wants to go shopping\")\n> 2. Show real merge marker contents, rather than e.g. \"yours:sample.txt\".\n> 3. Swap yours/theirs terms for source/target.\n> 4. General wordsmithing.\n>\n> Signed-off-by: Adam Johnson \n> ---\n> doc: merge: improve conflict presentation docs\n>\n> Improvements:\n>\n> 1. Remove the sexist example (\"Barbie... wants to go shopping\")\n> 2. Show real merge marker contents, rather than e.g.\n> \"yours:sample.txt\".\n> 3. Swap yours/theirs terms for source/target.\n> 4. General wordsmithing.\n>\n> Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1505%2Fadamchainz%2Faj%2Fmerge-conflict-docs-improvements-v1\n> Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1505/adamchainz/aj/merge-conflict-docs-improvements-v1\n> Pull-Request: https://github.com/git/git/pull/1505\n>\n> Documentation/git-merge.txt | 75 ++++++++++++++++++-------------------\n> 1 file changed, 37 insertions(+), 38 deletions(-)\n>\n> diff --git a/Documentation/git-merge.txt b/Documentation/git-merge.txt\n> index 0aeff572a59..b4523362e48 100644\n> --- a/Documentation/git-merge.txt\n> +++ b/Documentation/git-merge.txt\n> @@ -233,11 +233,11 @@ HOW CONFLICTS ARE PRESENTED\n>\n> During a merge, the working tree files are updated to reflect the result\n> of the merge. Among the changes made to the common ancestor's version,\n> -non-overlapping ones (that is, you changed an area of the file while the\n> -other side left that area intact, or vice versa) are incorporated in the\n> -final result verbatim. When both sides made changes to the same area,\n> +non-overlapping ones, where only one side changed an area, are incorporated\n> +in the final result as-is. When both sides made changes to the same area,\n> however, Git cannot randomly pick one side over the other, and asks you to\n> -resolve it by leaving what both sides did to that area.\n> +resolve it. Git adds changes from both sides, and optionally the\n> +original content from the common ancestor, surrounded by merge markers.\n\nEverywhere else in the manual we have referred to these as \"conflict\nmarkers\", not \"merge markers\". (Even if you spread out to the whole\ncodebase, there are only two places in the code (xdiff-interface.h and\nt6402) that refers to \"merge markers\", while there are 94 places that\nuse \"conflict markers\". We should fix those two to be like the other\n94.)\n\nOther than that, this rewrite seems good.\n\n> By default, Git uses the same style as the one used by the \"merge\" program\n> from the RCS suite to present such a conflicted hunk, like this:\n\nIf we're updating this documentation, perhaps it's time to drop the\nmention of RCS? 15 years ago, the reference made sense to bring up --\nthere were numerous folks who were new to Git that were familiar with\nRCS (even if only via CVS). The number of folks left who are still\naware of RCS but are not aware of Git, is probably very tiny, while\nthere are many folks new to Git who haven't heard of RCS.\n\nMaybe it still makes sense and doesn't hurt, but I'm not sure what\nit's buying us anymore.\n\n> @@ -245,71 +245,70 @@ from the RCS suite to present such a conflicted hunk, like this:\n> ------------\n> Here are lines that are either unchanged from the common\n> ancestor, or cleanly resolved because only one side changed,\n> -or cleanly resolved because both sides changed the same way.\n> -<<<<<<< yours:sample.txt\n> -Conflict resolution is hard;\n> -let's go shopping.\n> +or cleanly resolved because both sides changed identically.\n> +<<<<<<< HEAD\n> +Git makes conflict resolution straightforward.\n> =======\n> Git makes conflict resolution easy.\n> ->>>>>>> theirs:sample.txt\n> +>>>>>>> main\n> And here is another line that is cleanly resolved or unmodified.\n> ------------\n\nThree items here about the modifications to the conflict marker annotations:\n\n(1) The use of \"HEAD\": For a decade and a half, merges always involved\nmodifying the worktree as well, which somewhat forced merges to always\nbe done with HEAD. With a new merge backend, that is not necessary\nanymore, and conflicts can be generated (e.g. by `git log\n--remerge-diff` or `git replay` or `git merge-tree`) which do not\ninvolve HEAD. The use of \"HEAD\" and \"main\" might both be considered\nsuboptimal in the world we are moving into. For example, perhaps:\n\n <<<<<<< current_branch (\"summary of commit msg that current_branch\npoints to\")\n ...\n >>>>>>> main (\"summary of commit main points to\")\n\nwould be better, especially in cases where the user specified to merge\na hash rather than a branch name. (Note that this is similar to how\n--remerge-diff behaves.) Of course, I've started to turn this into a\nquestion of what we should change the implementation to, whereas you\nwere trying to document existing behavior, but sometimes documentation\nhighlights these subtle things...\n\n(2) Conflict markers carry an extra annotation of the form :FILENAME\nwhenever the filename differs on the two sides of history, so these\ncould be e.g. HEAD:sample.txt and main:renamed.txt in a real example.\nGranted, most conflicts probably don't involve renamed files, but\nwould it be more useful from a documentation standpoint to make users\naware of what might be seen?\n\n(3) The use of \"real\" conflict markers instead of sample ones, meant\nthat for the diff3 case below you used \"81821ce\" as the annotation.\nThe odds of a user having that particular hash is pretty slim, and\nfeels like much more of a distraction than using some synthetic hash\nor label. More on that below where it comes up.\n\n(All that said, I'm not sure what's best to use for the documentation\nright now, but thought these were relevant items to bring up.)\n\n> The area where a pair of conflicting changes happened is marked with markers\n> `<<<<<<<`, `=======`, and `>>>>>>>`. The part before the `=======`\n> -is typically your side, and the part afterwards is typically their side.\n> +is typically from the target (where you’re merging into), and the part\n> +afterwards is typically from the source (where you’re merging from).\n\nI dislike the whole \"yours\" & \"theirs\" thing. It would have been fine\nas originally designed, but in the UI at some point the terms were\nhardcoded and documented to stages 2 & 3 and rebase came along where\nthose meanings were exactly backwards. But, it's unfixable now, so\nthe best we can do is just to avoid these terms. I'm happy to see\nthem used one less place here.\n\n> +The default format does not show what the original version contained in the\n> +conflicting area. You cannot tell how many lines have been deleted and\n> +replaced on either side. The only thing you can tell is that the target side\n> +says \"straightforward\", while the source side says \"easy\".\n>\n> -The default format does not show what the original said in the conflicting\n> -area. You cannot tell how many lines are deleted and replaced with\n> -Barbie's remark on your side. The only thing you can tell is that your\n> -side wants to say it is hard and you'd prefer to go shopping, while the\n> -other side wants to claim it is easy.\n> +You can use an alternative conflict marker style by setting the\n> +`merge.conflictStyle` configuration variable to either \"diff3\" or \"zdiff3\".\n> +Both of these styles show the original version of the conflicted area, which\n> +may help you find a better resolution.\n>\n> -An alternative style can be used by setting the \"merge.conflictStyle\"\n> -configuration variable to either \"diff3\" or \"zdiff3\". In \"diff3\"\n> -style, the above conflict may look like this:\n> +In the \"diff3\" style, the above conflict looks like this:\n>\n> ------------\n> Here are lines that are either unchanged from the common\n> ancestor, or cleanly resolved because only one side changed,\n> -<<<<<<< yours:sample.txt\n> -or cleanly resolved because both sides changed the same way.\n> -Conflict resolution is hard;\n> -let's go shopping.\n> -||||||| base:sample.txt\n> +<<<<<<< HEAD\n> or cleanly resolved because both sides changed identically.\n> +Git makes conflict resolution straightforward.\n> +||||||| 81821ce\n\nHere's what I was referring to above. \"Using real values\" ends up\ncausing us to use a value (81821ce) that a user would basically never\nsee, and thus seems more likely to confuse them than using a\ndescriptive term or even a fake hash (such as \"aaaaaaa\").\n\n> +or cleanly resolved because both sides changed the same way.\n> Conflict resolution is hard.\n> =======\n> -or cleanly resolved because both sides changed the same way.\n> +or cleanly resolved because both sides changed identically.\n> Git makes conflict resolution easy.\n> ->>>>>>> theirs:sample.txt\n> +>>>>>>> main\n> And here is another line that is cleanly resolved or unmodified.\n> ------------\n>\n> -while in \"zdiff3\" style, it may look like this:\n> +while in the \"zdiff3\" style, it looks like this:\n\nThe insertion of \"the\" surprises me. It reads better without it to\nme, but I don't know if I'm just used to some grammar rule being\nviolated or something. Why is \"the\" necessary here?\n\n> ------------\n> Here are lines that are either unchanged from the common\n> ancestor, or cleanly resolved because only one side changed,\n> -or cleanly resolved because both sides changed the same way.\n> -<<<<<<< yours:sample.txt\n> -Conflict resolution is hard;\n> -let's go shopping.\n> -||||||| base:sample.txt\n> or cleanly resolved because both sides changed identically.\n> +<<<<<<< HEAD\n> +Git makes conflict resolution straightforward.\n> +||||||| 81821ce\n> +or cleanly resolved because both sides changed the same way.\n> Conflict resolution is hard.\n> =======\n> Git makes conflict resolution easy.\n> ->>>>>>> theirs:sample.txt\n> +>>>>>>> main\n> And here is another line that is cleanly resolved or unmodified.\n> ------------\n>\n> -In addition to the `<<<<<<<`, `=======`, and `>>>>>>>` markers, it uses\n> -another `|||||||` marker that is followed by the original text. You can\n> -tell that the original just stated a fact, and your side simply gave in to\n> -that statement and gave up, while the other side tried to have a more\n> -positive attitude. You can sometimes come up with a better resolution by\n> -viewing the original.\n> +The original commit SHA and text are shown after another marker, `|||||||`.\n> +This region lets you now see that both sides made the edit from \"the same\n> +way\" to \"identically\", as well as editing the following line. The \"diff3\"\n> +style keeps all changed lines within the markers, whilst the \"zdiff3\" style\n> +moves the commonly edited line before the marker.\n\nThe comment you added here about zdiff3 is correct in context, but\nfeels misleading in general. commonly edited lines at the beginning\nof the two sides are moved before the conflict markers, commonly\nedited lines at the ending of the two sides are moved after the\nmarkers, and commonly edited lines anywhere else are left where they\nare.\n"},{"id":"476732","messageId":"64582305ee966_4e612948f@chronos.notmuch","threadId":"59698","inReplyTo":"xmqqzg6ibe42.fsf@gitster.g","subject":"Re: [PATCH] doc: merge: improve conflict presentation docs","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2023-05-07T22:15:33Z","receivedAt":"2023-05-07T22:15:40Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"Junio C Hamano wrote:\n> \"Adam Johnson via GitGitGadget\" writes:\n> > From: Adam Johnson \n> >\n> > Improvements:\n> >\n> > 1. Remove the sexist example (\"Barbie... wants to go shopping\")\n> \n> \"Barbie goes shopping\" is a pretty common meme.\n\nAnd it's not sexist in the least.\n\nTony Montana likes big guns, and it's not sexist to say that outloud.\n\nTina can like big guns as well, and Peter can like to go shopping.\n\nThe fact that a preference happens to match a stereotype doesn't mean that\nsaying so constitutes bigotry.\n\nI'm Mexican and I like tacos. There's nothing wrong with somebody saying\n\"Felipe likes tacos\", especially because it's true.\n\nSo what if Barbie likes to go shopping? So what if Amanda likes the color pink?\n\nAm I not allowed to like tacos because I'm Mexican?\n\n-- \nFelipe Contreras\n"},{"id":"476733","messageId":"645825614027d_4e612949f@chronos.notmuch","threadId":"59698","inReplyTo":"CABPp-BE50neqaQbE2tTq_=fEM9j_8-cxgj7xKcSfAunoWjsigw@mail.gmail.com","subject":"Re: [PATCH] doc: merge: improve conflict presentation docs","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2023-05-07T22:25:37Z","receivedAt":"2023-05-07T22:26:03Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"Elijah Newren wrote:\n> On Fri, May 5, 2023 at 6:58 AM Adam Johnson via GitGitGadget\n> wrote:\n\n> > -while in \"zdiff3\" style, it may look like this:\n> > +while in the \"zdiff3\" style, it looks like this:\n> \n> The insertion of \"the\" surprises me. It reads better without it to\n> me, but I don't know if I'm just used to some grammar rule being\n> violated or something. Why is \"the\" necessary here?\n\nIt's not necessary, but the addition of the word \"the\" suggests a single\nwell-known term.\n\nIf you say \"minimalist style\", that can refer to a broad range of styles that\nare generally refered to as of \"minimalist\" type. On the other hand if you say\n\"the Minimalist tyle\" it evokes a *single* style that cannot be confused with\nany other.\n\nIn this case it's a single style, so \"the\" is warranted.\n\n-- \nFelipe Contreras"},{"id":"476747","messageId":"CABPp-BE4X=uk5E37KYWMZE5W-myRgXMj8fTpVZ7=FF05PoR5iQ@mail.gmail.com","threadId":"59698","inReplyTo":"CABPp-BE50neqaQbE2tTq_=fEM9j_8-cxgj7xKcSfAunoWjsigw@mail.gmail.com","subject":"Re: [PATCH] doc: merge: improve conflict presentation docs","fromName":"Elijah Newren","fromEmail":"newren@gmail.com","sentAt":"2023-05-08T15:25:04Z","receivedAt":"2023-05-08T15:25:21Z","isPatch":true,"sender":{"key":"newren@gmail.com","avatar":"https://avatars.githubusercontent.com/u/5455730?v=4"},"body":"On Sat, May 6, 2023 at 4:40 PM Elijah Newren wrote:\n>\n> On Fri, May 5, 2023 at 6:58 AM Adam Johnson via GitGitGadget\n> wrote:\n> >\n[...]\n> Everywhere else in the manual we have referred to these as \"conflict\n> markers\", not \"merge markers\".\n[...]\n> If we're updating this documentation, perhaps it's time to drop the\n> mention of RCS?\n[...]\n> I dislike the whole \"yours\" & \"theirs\" thing. [...] I'm happy to see\n> them used one less place here.\n[...]\n> \"Using real values\" [...] seems more likely to confuse them than using a\n> descriptive term or even a fake hash (such as \"aaaaaaa\").\n[...]\n> The comment you added here about zdiff3 is correct in context,\n[...]\n\nI realized I also failed to note that I like the patch in general and\nthink it has some good cleanups, even if there are potential further\nimprovements to make. Looking forward to your reroll.\n"}]}