[v3,0/4] cleanup revisions.txt

Message

Denton Liu May 5, 2019, 4:06 p.m. UTC

Thanks again for the comments, Andreas! I've incorporated all of them
into this reroll.

---

Changes since v2:

* Marked more optional arguments with []
* Added Andreas' "revisions.txt: remove ambibuity between <rev>:<path>
  and :<path>" patch

Changes since v1:

* Added patch to fix instances of "rev" to "<rev>"
* Marked all optional rev arguments with []


Andreas Heiduk (1):
  revisions.txt: remove ambibuity between <rev>:<path> and :<path>

Denton Liu (3):
  revisions.txt: change "rev" to "<rev>"
  revisions.txt: mark optional rev arguments with []
  revisions.txt: mention <rev>~ form

 Documentation/revisions.txt | 29 ++++++++++++++---------------
 1 file changed, 14 insertions(+), 15 deletions(-)

Comments

Denton Liu May 5, 2019, 4:10 p.m. UTC | #1

Sorry, I specified the wrong --in-reply-to. The correct parent message
should be this[1].

[1]: https://public-inbox.org/git/[e-mail]/

On Sun, May 05, 2019 at 12:06:54PM -0400, Denton Liu wrote:
> Thanks again for the comments, Andreas! I've incorporated all of them
> into this reroll.
> 
> ---
> 
> Changes since v2:
> 
> * Marked more optional arguments with []
> * Added Andreas' "revisions.txt: remove ambibuity between <rev>:<path>
>   and :<path>" patch
> 
> Changes since v1:
> 
> * Added patch to fix instances of "rev" to "<rev>"
> * Marked all optional rev arguments with []
> 
> 
> Andreas Heiduk (1):
>   revisions.txt: remove ambibuity between <rev>:<path> and :<path>
> 
> Denton Liu (3):
>   revisions.txt: change "rev" to "<rev>"
>   revisions.txt: mark optional rev arguments with []
>   revisions.txt: mention <rev>~ form
> 
>  Documentation/revisions.txt | 29 ++++++++++++++---------------
>  1 file changed, 14 insertions(+), 15 deletions(-)
> 
> -- 
> 2.21.0.1049.geb646f7864
>

Denton Liu May 7, 2019, 2:29 p.m. UTC | #2

Hi Junio,

I noticed that there are both `dl/rev-tilde-doc-clarify` and
`dl/revisions-doc-update` branches. The latter comes from this patchset
and is a complete replacement for the former.

Thanks,

Denton

On Sun, May 05, 2019 at 12:06:54PM -0400, Denton Liu wrote:
> Thanks again for the comments, Andreas! I've incorporated all of them
> into this reroll.
> 
> ---
> 
> Changes since v2:
> 
> * Marked more optional arguments with []
> * Added Andreas' "revisions.txt: remove ambibuity between <rev>:<path>
>   and :<path>" patch
> 
> Changes since v1:
> 
> * Added patch to fix instances of "rev" to "<rev>"
> * Marked all optional rev arguments with []
> 
> 
> Andreas Heiduk (1):
>   revisions.txt: remove ambibuity between <rev>:<path> and :<path>
> 
> Denton Liu (3):
>   revisions.txt: change "rev" to "<rev>"
>   revisions.txt: mark optional rev arguments with []
>   revisions.txt: mention <rev>~ form
> 
>  Documentation/revisions.txt | 29 ++++++++++++++---------------
>  1 file changed, 14 insertions(+), 15 deletions(-)
> 
> -- 
> 2.21.0.1049.geb646f7864
>

Junio C Hamano May 7, 2019, 3:16 p.m. UTC | #3

Denton Liu <[e-mail]> writes:

> I noticed that there are both `dl/rev-tilde-doc-clarify` and
> `dl/revisions-doc-update` branches. The latter comes from this patchset
> and is a complete replacement for the former.

Thanks for spotting.  Will replace the former with the updated
patches.

Thanks.