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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to