{"thread":{"id":"44665","subject":"[PATCH 0/4] doc: fixes to gitcore-tutorial.txt","startedAt":"2016-12-09T15:53:02Z","lastAt":"2016-12-13T21:48:43Z","messageCount":15,"participants":["Kristoffer Haugsbakk","Junio C Hamano","Philip Oakley"],"isPatch":true,"patchVersion":1,"patchTotal":4},"messages":[{"id":"307360","messageId":"20161209155112.2112-1-kristoffer.haugsbakk@gmail.com","threadId":"44665","inReplyTo":null,"subject":"[PATCH 0/4] doc: fixes to gitcore-tutorial.txt","fromName":"Kristoffer Haugsbakk","fromEmail":"kristoffer.haugsbakk@gmail.com","sentAt":"2016-12-09T15:51:08Z","receivedAt":"2016-12-09T15:53:02Z","isPatch":true,"sender":{"key":"kristoffer.haugsbakk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2229597?v=4"},"body":"This series of patches attempts to fix some minor mistakes in\ngitcore-tutorial.txt that I found while reading it. They are all\nconcerned with grammar and things like accidentally omitted words.\n\nI previously sent a single patch on 2016-11-04 (\"[PATCH] doc: fill in\nomitted word\"). The patch \"doc: omit needless \"for\"\" is a follow-up to\nthat, taking into consideration the feedback from Junio C Hamano and\nJeff King.\n\nKristoffer Haugsbakk (4):\n doc: add articles (grammar)\n doc: add verb in front of command to run\n doc: make the intent of sentence clearer\n doc: omit needless \"for\"\n\n Documentation/gitcore-tutorial.txt | 12 ++++++------\n 1 file changed, 6 insertions(+), 6 deletions(-)\n\n-- \n2.11.0\n\n"},{"id":"307361","messageId":"20161209155112.2112-2-kristoffer.haugsbakk@gmail.com","threadId":"44665","inReplyTo":"20161209155112.2112-1-kristoffer.haugsbakk@gmail.com","subject":"[PATCH 1/4] doc: add articles (grammar)","fromName":"Kristoffer Haugsbakk","fromEmail":"kristoffer.haugsbakk@gmail.com","sentAt":"2016-12-09T15:51:09Z","receivedAt":"2016-12-09T15:53:04Z","isPatch":true,"sender":{"key":"kristoffer.haugsbakk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2229597?v=4"},"body":"Add definite and indefinite articles in three places where they were\nmissing.\n\n- Use \"the\" in front of a directory name\n- Use \"the\" in front of \"style of cooperation\"\n- Use an indefinite article in front of \"CVS background\"\n\nSigned-off-by: Kristoffer Haugsbakk \n---\n Documentation/gitcore-tutorial.txt | 6 +++---\n 1 file changed, 3 insertions(+), 3 deletions(-)\n\ndiff --git a/Documentation/gitcore-tutorial.txt b/Documentation/gitcore-tutorial.txt\nindex 4546fa0d7..6c434aff3 100644\n--- a/Documentation/gitcore-tutorial.txt\n+++ b/Documentation/gitcore-tutorial.txt\n@@ -1368,7 +1368,7 @@ $ git repack\n will do it for you. If you followed the tutorial examples, you\n would have accumulated about 17 objects in `.git/objects/??/`\n directories by now. 'git repack' tells you how many objects it\n-packed, and stores the packed file in `.git/objects/pack`\n+packed, and stores the packed file in the `.git/objects/pack`\n directory.\n \n [NOTE]\n@@ -1543,9 +1543,9 @@ like this:\n Working with Others, Shared Repository Style\n --------------------------------------------\n \n-If you are coming from CVS background, the style of cooperation\n+If you are coming from a CVS background, the style of cooperation\n suggested in the previous section may be new to you. You do not\n-have to worry. Git supports \"shared public repository\" style of\n+have to worry. Git supports the \"shared public repository\" style of\n cooperation you are probably more familiar with as well.\n \n See linkgit:gitcvs-migration[7] for the details.\n-- \n2.11.0\n\n"},{"id":"307362","messageId":"20161209155112.2112-3-kristoffer.haugsbakk@gmail.com","threadId":"44665","inReplyTo":"20161209155112.2112-1-kristoffer.haugsbakk@gmail.com","subject":"[PATCH 2/4] doc: add verb in front of command to run","fromName":"Kristoffer Haugsbakk","fromEmail":"kristoffer.haugsbakk@gmail.com","sentAt":"2016-12-09T15:51:10Z","receivedAt":"2016-12-09T15:53:14Z","isPatch":true,"sender":{"key":"kristoffer.haugsbakk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2229597?v=4"},"body":"Instead of using the command 'git clone' as a verb, use \"run\" as the\nverb indicating the action of executing the command 'git clone'.\n\nSigned-off-by: Kristoffer Haugsbakk \n---\n Documentation/gitcore-tutorial.txt | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/Documentation/gitcore-tutorial.txt b/Documentation/gitcore-tutorial.txt\nindex 6c434aff3..72ed90ca3 100644\n--- a/Documentation/gitcore-tutorial.txt\n+++ b/Documentation/gitcore-tutorial.txt\n@@ -1478,7 +1478,7 @@ You can repack this private repository whenever you feel like.\n A recommended work cycle for a \"subsystem maintainer\" who works\n on that project and has an own \"public repository\" goes like this:\n \n-1. Prepare your work repository, by 'git clone' the public\n+1. Prepare your work repository, by running 'git clone' on the public\n repository of the \"project lead\". The URL used for the\n initial cloning is stored in the remote.origin.url\n configuration variable.\n-- \n2.11.0\n\n"},{"id":"307363","messageId":"20161209155112.2112-4-kristoffer.haugsbakk@gmail.com","threadId":"44665","inReplyTo":"20161209155112.2112-1-kristoffer.haugsbakk@gmail.com","subject":"[PATCH 3/4] doc: make the intent of sentence clearer","fromName":"Kristoffer Haugsbakk","fromEmail":"kristoffer.haugsbakk@gmail.com","sentAt":"2016-12-09T15:51:11Z","receivedAt":"2016-12-09T15:53:17Z","isPatch":true,"sender":{"key":"kristoffer.haugsbakk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2229597?v=4"},"body":"By adding the word \"just\", which might have been accidentally omitted.\n\nAdding the word \"just\" makes it clear that the point is to *not* do an\noctopus merge simply because you *can* do it. In other words, you\nshould have a reason for doing it beyond simply having two (seemingly)\nindependent commits that you need to merge into another branch, since\nit's not always the best approach.\n\nThe previous sentence made it look more like it was trying to say that\nyou shouldn't do an octopus merge *because* you can do an octopus merge.\nAlthough this interpretation doesn't make sense and the rest of the\nparagraph makes the intended meaning clear, this adjustment should make\nthe intent of the sentence more immediately clear to the reader.\n\nSigned-off-by: Kristoffer Haugsbakk \n---\n Documentation/gitcore-tutorial.txt | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/Documentation/gitcore-tutorial.txt b/Documentation/gitcore-tutorial.txt\nindex 72ed90ca3..72ca9c1ef 100644\n--- a/Documentation/gitcore-tutorial.txt\n+++ b/Documentation/gitcore-tutorial.txt\n@@ -1635,7 +1635,7 @@ $ git show-branch\n ++* [master~2] Pretty-print messages.\n ------------\n \n-Note that you should not do Octopus because you can. An octopus\n+Note that you should not do Octopus just because you can. An octopus\n is a valid thing to do and often makes it easier to view the\n commit history if you are merging more than two independent\n changes at the same time. However, if you have merge conflicts\n-- \n2.11.0\n\n"},{"id":"307364","messageId":"20161209155112.2112-5-kristoffer.haugsbakk@gmail.com","threadId":"44665","inReplyTo":"20161209155112.2112-1-kristoffer.haugsbakk@gmail.com","subject":"[PATCH 4/4] doc: omit needless \"for\"","fromName":"Kristoffer Haugsbakk","fromEmail":"kristoffer.haugsbakk@gmail.com","sentAt":"2016-12-09T15:51:12Z","receivedAt":"2016-12-09T15:53:18Z","isPatch":true,"sender":{"key":"kristoffer.haugsbakk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2229597?v=4"},"body":"What was intended was perhaps \"... plumbing does for you\" (\"you\" added), but\nsimply omitting the word \"for\" is more terse and gets the intended point across\njust as well, if not more so.\n\nI originally went with the approach of writing \"for you\", but Junio C\nHamano suggested this approach instead.\n\nSigned-off-by: Kristoffer Haugsbakk \n---\n\nNotes (kristoffers):\n The original patch was sent to the mailing list on 2016-11-04, and Junio\n replied with his suggested correction on 2016-11-10; see the cover\n letter.\n\n Documentation/gitcore-tutorial.txt | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/Documentation/gitcore-tutorial.txt b/Documentation/gitcore-tutorial.txt\nindex 72ca9c1ef..22309cfb4 100644\n--- a/Documentation/gitcore-tutorial.txt\n+++ b/Documentation/gitcore-tutorial.txt\n@@ -25,7 +25,7 @@ you want to understand Git's internals.\n The core Git is often called \"plumbing\", with the prettier user\n interfaces on top of it called \"porcelain\". You may not want to use the\n plumbing directly very often, but it can be good to know what the\n-plumbing does for when the porcelain isn't flushing.\n+plumbing does when the porcelain isn't flushing.\n \n Back when this document was originally written, many porcelain\n commands were shell scripts. For simplicity, it still uses them as\n-- \n2.11.0\n\n"},{"id":"307405","messageId":"xmqq1sxgr90f.fsf@gitster.mtv.corp.google.com","threadId":"44665","inReplyTo":"20161209155112.2112-1-kristoffer.haugsbakk@gmail.com","subject":"Re: [PATCH 0/4] doc: fixes to gitcore-tutorial.txt","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2016-12-09T23:13:36Z","receivedAt":"2016-12-09T23:13:43Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Kristoffer Haugsbakk writes:\n\n> This series of patches attempts to fix some minor mistakes in\n> gitcore-tutorial.txt that I found while reading it. They are all\n> concerned with grammar and things like accidentally omitted words.\n\nGrammar is not my forte, so even though I'll queue them as-is\nbecause I didn't spot anything wrong in them and thought they are\nall improvements, I'd appreciate somebody else to lend an eye or two\nover them.\n\nThanks.\n"},{"id":"307448","messageId":"88A192B34B3D4DDDA47628687AE458D3@PhilipOakley","threadId":"44665","inReplyTo":"20161209155112.2112-2-kristoffer.haugsbakk@gmail.com","subject":"Re: [PATCH 1/4] doc: add articles (grammar)","fromName":"Philip Oakley","fromEmail":"philipoakley@iee.org","sentAt":null,"receivedAt":"2016-12-10T20:28:35Z","isPatch":true,"sender":{"key":"philipoakley@iee.email","avatar":"https://avatars.githubusercontent.com/u/914343?v=4"},"body":"From: \"Kristoffer Haugsbakk\" \nSent: Friday, December 09, 2016 3:51 PM\n> Add definite and indefinite articles in three places where they were\n> missing.\n>\n> - Use \"the\" in front of a directory name\n> - Use \"the\" in front of \"style of cooperation\"\n> - Use an indefinite article in front of \"CVS background\"\n>\n> Signed-off-by: Kristoffer Haugsbakk \n> ---\n> Documentation/gitcore-tutorial.txt | 6 +++---\n> 1 file changed, 3 insertions(+), 3 deletions(-)\n>\n> diff --git a/Documentation/gitcore-tutorial.txt \n> b/Documentation/gitcore-tutorial.txt\n> index 4546fa0d7..6c434aff3 100644\n> --- a/Documentation/gitcore-tutorial.txt\n> +++ b/Documentation/gitcore-tutorial.txt\n> @@ -1368,7 +1368,7 @@ $ git repack\n> will do it for you. If you followed the tutorial examples, you\n> would have accumulated about 17 objects in `.git/objects/??/`\n> directories by now. 'git repack' tells you how many objects it\n> -packed, and stores the packed file in `.git/objects/pack`\n> +packed, and stores the packed file in the `.git/objects/pack`\n> directory.\n>\n> [NOTE]\n> @@ -1543,9 +1543,9 @@ like this:\n> Working with Others, Shared Repository Style\n> --------------------------------------------\n>\n> -If you are coming from CVS background, the style of cooperation\n> +If you are coming from a CVS background, the style of cooperation\n> suggested in the previous section may be new to you. You do not\n> -have to worry. Git supports \"shared public repository\" style of\n> +have to worry. Git supports the \"shared public repository\" style of\n> cooperation you are probably more familiar with as well.\n>\n> See linkgit:gitcvs-migration[7] for the details.\n> -- \n> 2.11.0\n\nThis looks good to me.\n--\nPhilip (UK) \n\n"},{"id":"307449","messageId":"2C16DAFED0D94B28AF9CE7F8BBA122F0@PhilipOakley","threadId":"44665","inReplyTo":"20161209155112.2112-3-kristoffer.haugsbakk@gmail.com","subject":"Re: [PATCH 2/4] doc: add verb in front of command to run","fromName":"Philip Oakley","fromEmail":"philipoakley@iee.org","sentAt":null,"receivedAt":"2016-12-10T20:48:48Z","isPatch":true,"sender":{"key":"philipoakley@iee.email","avatar":"https://avatars.githubusercontent.com/u/914343?v=4"},"body":"From: \"Kristoffer Haugsbakk\" \nSent: Friday, December 09, 2016 3:51 PM\n> Instead of using the command 'git clone' as a verb, use \"run\" as the\n> verb indicating the action of executing the command 'git clone'.\n\nI would expect 'cloning' as the commonly in use verb here, with the command \nthen quoted.\n\n>\n> Signed-off-by: Kristoffer Haugsbakk \n> ---\n> Documentation/gitcore-tutorial.txt | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n>\n> diff --git a/Documentation/gitcore-tutorial.txt \n> b/Documentation/gitcore-tutorial.txt\n> index 6c434aff3..72ed90ca3 100644\n> --- a/Documentation/gitcore-tutorial.txt\n> +++ b/Documentation/gitcore-tutorial.txt\n> @@ -1478,7 +1478,7 @@ You can repack this private repository whenever you \n> feel like.\n> A recommended work cycle for a \"subsystem maintainer\" who works\n> on that project and has an own \"public repository\" goes like this:\n>\n> -1. Prepare your work repository, by 'git clone' the public\n> +1. Prepare your work repository, by running 'git clone' on the public\n\nPerhaps ?\n... by cloning ('git clone ') the public\n\nThe full command is shown using the same terminology as the following line.\n\n> repository of the \"project lead\". The URL used for the\n> initial cloning is stored in the remote.origin.url\n> configuration variable.\n> -- \n> 2.11.0\n>\n>\n--\nPhilip \n\n"},{"id":"307450","messageId":"34DFA15DE05248069167FE613B66CD70@PhilipOakley","threadId":"44665","inReplyTo":"20161209155112.2112-4-kristoffer.haugsbakk@gmail.com","subject":"Re: [PATCH 3/4] doc: make the intent of sentence clearer","fromName":"Philip Oakley","fromEmail":"philipoakley@iee.org","sentAt":null,"receivedAt":"2016-12-10T20:52:23Z","isPatch":true,"sender":{"key":"philipoakley@iee.email","avatar":"https://avatars.githubusercontent.com/u/914343?v=4"},"body":"From: \"Kristoffer Haugsbakk\" \nSent: Friday, December 09, 2016 3:51 PM\n> By adding the word \"just\", which might have been accidentally omitted.\n>\n> Adding the word \"just\" makes it clear that the point is to *not* do an\n> octopus merge simply because you *can* do it. In other words, you\n> should have a reason for doing it beyond simply having two (seemingly)\n> independent commits that you need to merge into another branch, since\n> it's not always the best approach.\n>\n> The previous sentence made it look more like it was trying to say that\n> you shouldn't do an octopus merge *because* you can do an octopus merge.\n> Although this interpretation doesn't make sense and the rest of the\n> paragraph makes the intended meaning clear, this adjustment should make\n> the intent of the sentence more immediately clear to the reader.\n>\n> Signed-off-by: Kristoffer Haugsbakk \n> ---\n> Documentation/gitcore-tutorial.txt | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n>\n> diff --git a/Documentation/gitcore-tutorial.txt \n> b/Documentation/gitcore-tutorial.txt\n> index 72ed90ca3..72ca9c1ef 100644\n> --- a/Documentation/gitcore-tutorial.txt\n> +++ b/Documentation/gitcore-tutorial.txt\n> @@ -1635,7 +1635,7 @@ $ git show-branch\n> ++* [master~2] Pretty-print messages.\n> ------------\n>\n> -Note that you should not do Octopus because you can. An octopus\n> +Note that you should not do Octopus just because you can. An octopus\n> is a valid thing to do and often makes it easier to view the\n> commit history if you are merging more than two independent\n> changes at the same time. However, if you have merge conflicts\n> -- \n> 2.11.0\n>\nLooks like a reasonable emphasis to me\n--\nPhilip (UK) \n\n"},{"id":"307466","messageId":"2645548666054ED5BD30436E9DA41C14@PhilipOakley","threadId":"44665","inReplyTo":"20161209155112.2112-5-kristoffer.haugsbakk@gmail.com","subject":"Re: [PATCH 4/4] doc: omit needless \"for\"","fromName":"Philip Oakley","fromEmail":"philipoakley@iee.org","sentAt":null,"receivedAt":"2016-12-11T13:05:01Z","isPatch":true,"sender":{"key":"philipoakley@iee.email","avatar":"https://avatars.githubusercontent.com/u/914343?v=4"},"body":"From: \"Kristoffer Haugsbakk\" \nSent: Friday, December 09, 2016 3:51 PM\n> What was intended was perhaps \"... plumbing does for you\" (\"you\" added),\n> but\n> simply omitting the word \"for\" is more terse and gets the intended point\n> across\n> just as well, if not more so.\n\nAfter some thought, I think the original is more 'right'.\n\nWithout the 'for' it suggests that understanding individual plumbing \ncommands would explain some issue being seen with a fancy porcelain command \nwhich they probably don't. Rather the 'for' is forward looking toward using \nthe plumbing commands as tools to investigate and then re-plumb the \naestehetics to the desired output.\n\nThe whole porcelain euphemism makes for some awkward phrasing.\n\n>\n> I originally went with the approach of writing \"for you\", but Junio C\n> Hamano suggested this approach instead.\n>\n> Signed-off-by: Kristoffer Haugsbakk \n> ---\n>\n> Notes (kristoffers):\n> The original patch was sent to the mailing list on 2016-11-04, and\n> Junio\n> replied with his suggested correction on 2016-11-10; see the cover\n> letter.\n>\n> Documentation/gitcore-tutorial.txt | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n>\n> diff --git a/Documentation/gitcore-tutorial.txt\n> b/Documentation/gitcore-tutorial.txt\n> index 72ca9c1ef..22309cfb4 100644\n> --- a/Documentation/gitcore-tutorial.txt\n> +++ b/Documentation/gitcore-tutorial.txt\n> @@ -25,7 +25,7 @@ you want to understand Git's internals.\n> The core Git is often called \"plumbing\", with the prettier user\n\nIf we are tidying up here, then perhaps\ns/core Git is often/Git commands are often/\nto better clarify what aspects are plumbing / porcelain.\n\n> interfaces on top of it called \"porcelain\". You may not want to use the\n> plumbing directly very often, but it can be good to know what the\n> -plumbing does for when the porcelain isn't flushing.\n> +plumbing does when the porcelain isn't flushing.\n\nI'm not so sure that the direct allusion to 'flushing' is exactly the right\ntone. Part of the issue is the 'porcelain' is the initial euphemism. The \nother part is that both porcelain and plumbing commands have the same level \nof CLI un-prettiness, so the distinction isn't there.\n\nIn the end I strung together:\n\"\nThe core Git commands are often called \"plumbing\",\n while those with the prettier user friendly\n output are called \"porcelain\".\n\n You may not want to use the plumbing directly very often,\n but it can be good to know what the plumbing does\n when either the porcelain isn't flushing, or different output aethetics are \ndesired.\n\"\n\nThough having both prettier and friendly in the same phrase maybe overkill.\n\n>\n> Back when this document was originally written, many porcelain\n> commands were shell scripts. For simplicity, it still uses them as\n> -- \n> 2.11.0\n>\n--\nPhilip\n\n"},{"id":"307468","messageId":"C42D5B32E3FC43F8933D3F66CB2456C7@PhilipOakley","threadId":"44665","inReplyTo":"2645548666054ED5BD30436E9DA41C14@PhilipOakley","subject":"Re: [PATCH 4/4] doc: omit needless \"for\"","fromName":"Philip Oakley","fromEmail":"philipoakley@iee.org","sentAt":null,"receivedAt":"2016-12-11T14:09:05Z","isPatch":true,"sender":{"key":"philipoakley@iee.email","avatar":"https://avatars.githubusercontent.com/u/914343?v=4"},"body":"From: \"Philip Oakley\" \n\n>> diff --git a/Documentation/gitcore-tutorial.txt\n>> b/Documentation/gitcore-tutorial.txt\n>> index 72ca9c1ef..22309cfb4 100644\n>> --- a/Documentation/gitcore-tutorial.txt\n>> +++ b/Documentation/gitcore-tutorial.txt\n>> @@ -25,7 +25,7 @@ you want to understand Git's internals.\n>> The core Git is often called \"plumbing\", with the prettier user\n> \n> If we are tidying up here, then perhaps\n> s/core Git is often/Git commands are often/\n> to better clarify what aspects are plumbing / porcelain.\n> \naccidentally dropped a word. Should have said\ns/core Git is often/core Git commands are often/\n\n"},{"id":"307594","messageId":"8737hsj7wp.fsf@gmail.com","threadId":"44665","inReplyTo":"88A192B34B3D4DDDA47628687AE458D3@PhilipOakley","subject":"Re: [PATCH 1/4] doc: add articles (grammar)","fromName":"Kristoffer Haugsbakk","fromEmail":"kristoffer.haugsbakk@gmail.com","sentAt":"2016-12-13T13:05:58Z","receivedAt":"2016-12-13T13:06:09Z","isPatch":true,"sender":{"key":"kristoffer.haugsbakk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2229597?v=4"},"body":"\nThank you for reviewing this series, Philip.\n\nPhilip Oakley writes:\n\n> This looks good to me.\n\nI'll add this header:\n\n Acked-by: Philip Oakley \n\nTo the commit message of this patch in the next review round (version 2\nof the patch series).\n\nLet me know if I should add another header, or do something different\nthan this.\n\n-- \nKristoffer Haugsbakk\n"},{"id":"307595","messageId":"871sxcj72y.fsf@gmail.com","threadId":"44665","inReplyTo":"34DFA15DE05248069167FE613B66CD70@PhilipOakley","subject":"Re: [PATCH 3/4] doc: make the intent of sentence clearer","fromName":"Kristoffer Haugsbakk","fromEmail":"kristoffer.haugsbakk@gmail.com","sentAt":"2016-12-13T13:23:49Z","receivedAt":"2016-12-13T13:23:56Z","isPatch":true,"sender":{"key":"kristoffer.haugsbakk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2229597?v=4"},"body":"\nPhilip Oakley writes:\n\n> Looks like a reasonable emphasis to me\n\nI'll add this header:\n\n Acked-by: Philip Oakley \n\nTo the commit message of this patch in the next review round (version 2\nof the patch series).\n\nLet me know if I should add another header, or do something different\nthan this.\n\n-- \nKristoffer Haugsbakk\n"},{"id":"307647","messageId":"xmqqoa0f4s2v.fsf@gitster.mtv.corp.google.com","threadId":"44665","inReplyTo":"8737hsj7wp.fsf@gmail.com","subject":"Re: [PATCH 1/4] doc: add articles (grammar)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2016-12-13T18:11:36Z","receivedAt":"2016-12-13T18:11:43Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Kristoffer Haugsbakk writes:\n\n> Thank you for reviewing this series, Philip.\n>\n> Philip Oakley writes:\n>\n>> This looks good to me.\n>\n> I'll add this header:\n>\n> Acked-by: Philip Oakley \n>\n> To the commit message of this patch in the next review round (version 2\n> of the patch series).\n>\n> Let me know if I should add another header, or do something different\n> than this.\n\nI was planning to merge all four from you as-is to 'next' today,\nthough. Should I wait?\n\n"},{"id":"307691","messageId":"87y3zjikdd.fsf@gmail.com","threadId":"44665","inReplyTo":"xmqqoa0f4s2v.fsf@gitster.mtv.corp.google.com","subject":"Re: [PATCH 1/4] doc: add articles (grammar)","fromName":"Kristoffer Haugsbakk","fromEmail":"kristoffer.haugsbakk@gmail.com","sentAt":"2016-12-13T21:34:22Z","receivedAt":"2016-12-13T21:48:43Z","isPatch":true,"sender":{"key":"kristoffer.haugsbakk@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2229597?v=4"},"body":"\nJunio C Hamano writes:\n\n> I was planning to merge all four from you as-is to 'next' today,\n> though. Should I wait?\n\nI'll definitely defer to whatever you think is best. I guess it depends\non whether you are interested in Philip Oakley's suggestions. I sent\nthose emails to inform about what I intended to do in the next round, if\nit got to that point, since I haven't tried to contribute to such an\norganised project before. So I was informing about my assumptions about\nhow to deal with \"looks good to me\"-kinds of feedback.\n\nSo for my part, I'm happy with iterating on this (perhaps just adding\nthe two \"acks\", or also replying to the suggestions), or just merging it\nas-is.\n\n-- \nKristoffer Haugsbakk\n"}]}