Re: [PATCH v1] git-p4: fix AsciiDoc formatting

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> On 15 Mar 2016, at 09:38, Luke Diamand <luke@xxxxxxxxxxx> wrote:
> 
> On 14 March 2016 at 03:51, Junio C Hamano <gitster@xxxxxxxxx> wrote:
>> Lars Schneider <larsxschneider@xxxxxxxxx> writes:
>> 
>>>> diff --git a/Documentation/git-p4.txt b/Documentation/git-p4.txt
>>>> index 738cfde..140fc12 100644
>>>> --- a/Documentation/git-p4.txt
>>>> +++ b/Documentation/git-p4.txt
>>>> @@ -528,7 +528,7 @@ git-p4.largeFileSystem::
>>>> git config       git-p4.largeFileSystem GitLFS
>>>> -------------
>>>> +
>>>> -    [1] https://git-lfs.github.com/
>>>> +[1] https://git-lfs.github.com/
>>>> 
>>>> git-p4.largeFileExtensions::
>>>>     All files matching a file extension in the list will be processed
>>>> --
>>>> 2.5.1
>>>> 
>>> 
>>> Is there any chance to get this in? The formatting error is quite obvious here:
>>> https://git-scm.com/docs/git-p4
>>> 
>>> (while looking at this website I also noticed wrong double quotes around cp1252
>>> which I will fix with a subsequent patch)
>> 
>> I ignored this when I saw it first, primarily because I assumed that
>> anything related to git-p4 would get a review by somebody better
>> versed with git-p4 before I get to it; I forgot about the patch
>> after it was buried in the other messages without anything
>> happening.
>> 
>> Will take a look.
> 
> Comparing with the git-bisect-lk2009 page, it uses a slightly
> different approach for external HTTP references. The references are
> put at the end, as internal hyperlinks, and in a bulleted list. It
> would make sense to try to be consistent with that, unless there's a
> good reason not to.

Right! Thanks for the hint! I will post a reroll soonish.

Thanks,
Lars--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]