Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
From: kant2002 <git@git.apache.org>
To: dev@cordova.apache.org
Reply-To: dev@cordova.apache.org
Message-ID: <git-pr-236-cordova-docs@git.apache.org>
Subject: [GitHub] cordova-docs pull request: CB-6751 Added joDoc dependency
Content-Type: text/plain
Date: Sat, 13 Sep 2014 16:10:32 +0000 (UTC)

GitHub user kant2002 opened a pull request:

    https://github.com/apache/cordova-docs/pull/236

    CB-6751 Added joDoc dependency

    I try using NodeJS joDoc, but have following findings:
    1. It fail to work on Windows out of the box. Because it does not know how to create nested directory structure. Has to put fix for that. Fix available at https://github.com/kant2002/jodoc-js
    2. All pre- and post-processing logic should be reimplemented in JS.
    3. Should be found alternative for Nocogiri in Node.JS community. Seems that this link http://stackoverflow.com/questions/7977945/html-parser-on-nodejs has good options. 
    From first sight https://github.com/cheeriojs/cheerio looks best choice which allows finish implementation faster.
    jsdom looks good to, but looks like it is pain to setup on Windows. Since I develop on Windows, my preference is not to use it.
    
    Also joDocs looks not maintained anymore, so we probable have to put some effort to keep it running as needed for the Cordova. 
    
    Will push updates about this issue to this branch/pull request, as more progress will be done

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/kant2002/cordova-docs CB-6751

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cordova-docs/pull/236.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #236
    
----
commit 809c23b91772523570f6d8f5ca15bb41d6294ea1
Author: Andrey Kurdyumov <kant2002@gmail.com>
Date:   2014-09-13T16:05:08Z

    Added joDoc dependency

----


---
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.
---