Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
From: jsoref <git@git.apache.org>
To: dev@cordova.apache.org
Reply-To: dev@cordova.apache.org
References: <git-pr-41-cordova-plugin-file@git.apache.org>
In-Reply-To: <git-pr-41-cordova-plugin-file@git.apache.org>
Subject: [GitHub] cordova-plugin-file pull request: Update index.md
Content-Type: text/plain
Message-Id: <20140425151830.6FFF09516EA@tyr.zones.apache.org>
Date: Fri, 25 Apr 2014 15:18:30 +0000 (UTC)

Github user jsoref commented on a diff in the pull request:

    https://github.com/apache/cordova-plugin-file/pull/41#discussion_r12001097
  
    --- Diff: doc/index.md ---
    @@ -24,6 +24,20 @@ to HTML5 Rocks' [FileSystem article](http://www.html5rocks.com/en/tutorials/file
     on the subject. For an overview of other storage options, refer to Cordova's
     [storage guide](http://cordova.apache.org/docs/en/edge/cordova_storage_storage.md.html).
     
    +Over the standard file systems (`LocalFileSystem.PERSISTENCE` and `LocalFileSystem.TEMPORARY`) the cordova file plug-in allows access to further file systems. To use them, one needs to configure which are allowed for the application (see section configuration). To access a file within one of these file systems, use the following command:
    --- End diff --
    
    **Cordova** should be uppercase here.
    Also, **File Plug-in** probably should be too.
    
    I'm not sure about conventions for long lines, but based on 24+25, it looks like you should wrap at around 80 cols.
    
    if `configuration` is a section, it probably should have *some* markdown. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---