lkml.org 
[lkml]   [2018]   [Dec]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/2] Documentation: convert path-lookup from markdown to resturctured text
On Wed, 05 Dec 2018 10:02:51 +1100
NeilBrown <neilb@suse.com> wrote:

> This allows the document to be integrated with the main documentation
> tree.
> Changes include:
> - rename from .md to .rst
> - use `` for code, not single `
> - use correct sub-section marking
> - fix indented blocks, both code and non-code
> - fix external-link markup
>
> Signed-off-by: NeilBrown <neilb@suse.com>

Looks good at a first glance, but I have one question...

> Documentation/filesystems/index.rst | 2 +
> .../{path-lookup.md => path-lookup.rst} | 875 +++++++++++----------
> 2 files changed, 448 insertions(+), 429 deletions(-)
> rename Documentation/filesystems/{path-lookup.md => path-lookup.rst} (59%)
>
> diff --git a/Documentation/filesystems/index.rst b/Documentation/filesystems/index.rst
> index 46d1b1be3a51..3c0f441a9536 100644
> --- a/Documentation/filesystems/index.rst
> +++ b/Documentation/filesystems/index.rst
> @@ -359,3 +359,5 @@ encryption of files and directories.
> :maxdepth: 2
>
> fscrypt
> +
> +.. include:: path-lookup.rst

Is there a reason you used "include" rather than just adding it to the
TOC tree with the rest?

Thanks,

jon

\
 
 \ /
  Last update: 2018-12-05 20:46    [W:0.176 / U:0.128 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site