[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [PATCH] Update documentation for :exports (applicable to blocks
From: |
John Hendy |
Subject: |
Re: [O] [PATCH] Update documentation for :exports (applicable to blocks only, not inline) |
Date: |
Fri, 14 Mar 2014 11:06:49 -0500 |
On Fri, Mar 14, 2014 at 10:51 AM, Bastien <address@hidden> wrote:
> Hi John,
>
> thanks for the patch -- some comments below.
>
> John Hendy <address@hidden> writes:
>
>> Header arguments: clarify that `:exports' is only applicable to
>> blocks, not inline code
>
> This first line should include "org.texi:" to tell that the change
> happens in org.texi. See other commits for examples.
>
Gotcha. That wasn't in the manual example for the first line. I've
added it to my re-attempt attached.
>> * doc/org.texi (Exporting code blocks): add clarification; relevant for
>> blocks ony.
>
> Start with a capitalized letter: "Add clarification."
> Don't use semi-column. "Clarify" is enough IMO.
>
Even after reading the instructions, I *still* missed the capitalization bit...
>> * doc/org.texi (exports): add clarification; relevant for blocks only.
>
> When two changes get the same description use this:
>
> * doc/org.texi (Exporting code blocks, exports): Clarify.
>
Awesome. I wondered about that as well.
> In general, you can make it easier for you by reviewing the patch (hit
> `C-x v =' in the modified org.texi buffer), and then hit `C-x 4 a' on
> each change to create a Changelog buffer that you can safely edit.
>
Thanks for the tips.
>> The documentation change resulted from the expectation that
>> `:exports' could be used with inline code blocks, which is not the
>> case. Clarification was added to avoid future confusion on this
>> Babel block specific header argument.
>
> Please use auto-fill and fill-column to something between 72 and 80.
>
Done.
>> TINYCHANGE
>
> Thanks again for the patch, HTH,
>
No problem.
John
> --
> Bastien
0001-clarified-documentation-for-exports-only-applicable-.patch
Description: Text Data
- [O] [PATCH] Update documentation for :exports (applicable to blocks only, not inline), John Hendy, 2014/03/11
- Re: [O] [PATCH] Update documentation for :exports (applicable to blocks only, not inline), Bastien, 2014/03/12
- Re: [O] [PATCH] Update documentation for :exports (applicable to blocks only, not inline), John Hendy, 2014/03/12
- Re: [O] [PATCH] Update documentation for :exports (applicable to blocks only, not inline), Bastien, 2014/03/13
- Re: [O] [PATCH] Update documentation for :exports (applicable to blocks only, not inline), John Hendy, 2014/03/14
- Re: [O] [PATCH] Update documentation for :exports (applicable to blocks only, not inline), Bastien, 2014/03/14
- Re: [O] [PATCH] Update documentation for :exports (applicable to blocks only, not inline),
John Hendy <=
- Re: [O] [PATCH] Update documentation for :exports (applicable to blocks only, not inline), Bastien, 2014/03/14
- Re: [O] [PATCH] Update documentation for :exports (applicable to blocks only, not inline), John Hendy, 2014/03/14