[GitHub] cordova-docs pull request: Removing keyword index.

2016-02-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-docs/pull/508 --- 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

[GitHub] cordova-docs pull request: Removing keyword index.

2016-02-19 Thread nikhilkh
Github user nikhilkh commented on the pull request: https://github.com/apache/cordova-docs/pull/508#issuecomment-186334602 LGTM --- 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

[GitHub] cordova-docs pull request: Removing keyword index.

2016-02-18 Thread dblotsky
Github user dblotsky commented on the pull request: https://github.com/apache/cordova-docs/pull/508#issuecomment-186014176 @stevengill just removing code that's not needed anymore. The search box at the top basically obsoletes it. --- If your project is set up for it, you can reply

[GitHub] cordova-docs pull request: Removing keyword index.

2016-02-18 Thread rakatyal
Github user rakatyal commented on the pull request: https://github.com/apache/cordova-docs/pull/508#issuecomment-185993325 LGTM. --- 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

[GitHub] cordova-docs pull request: Removing keyword index.

2016-02-18 Thread stevengill
Github user stevengill commented on the pull request: https://github.com/apache/cordova-docs/pull/508#issuecomment-185991371 Looks fine to me. I don't know the context behind the change though. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cordova-docs pull request: Removing keyword index.

2016-02-18 Thread dblotsky
GitHub user dblotsky opened a pull request: https://github.com/apache/cordova-docs/pull/508 Removing keyword index. It has been replaced by search. You can merge this pull request into a Git repository by running: $ git pull https://github.com/MSOpenTech/cordova-docs

[GitHub] cordova-docs pull request: Removing keyword index.

2016-02-18 Thread dblotsky
Github user dblotsky commented on the pull request: https://github.com/apache/cordova-docs/pull/508#issuecomment-185980598 @rakatyal @stevengill please review when you have a moment. --- If your project is set up for it, you can reply to this email and have your reply appear on