[GitHub] cordova-docs pull request: CB-6680 Clarify instructions for instal...

2014-05-22 Thread cmarcelk
Github user cmarcelk closed the pull request at: https://github.com/apache/cordova-docs/pull/205 --- 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: CB-6680 Clarify instructions for instal...

2014-05-22 Thread clelland
Github user clelland commented on a diff in the pull request: https://github.com/apache/cordova-docs/pull/205#discussion_r12952956 --- Diff: docs/en/edge/guide/cli/index.md --- @@ -73,22 +74,41 @@ To install the `cordova` command-line tool, follow these steps: the CLI does

[GitHub] cordova-docs pull request: CB-6680 Clarify instructions for instal...

2014-05-21 Thread jsoref
Github user jsoref commented on a diff in the pull request: https://github.com/apache/cordova-docs/pull/205#discussion_r12906784 --- Diff: docs/en/edge/guide/cli/index.md --- @@ -73,22 +74,41 @@ To install the `cordova` command-line tool, follow these steps: the CLI does us

[GitHub] cordova-docs pull request: CB-6680 Clarify instructions for instal...

2014-05-21 Thread jsoref
Github user jsoref commented on the pull request: https://github.com/apache/cordova-docs/pull/205#issuecomment-43782968 I like this (I don't usually spend time commenting on things I don't like...) --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cordova-docs pull request: CB-6680 Clarify instructions for instal...

2014-05-21 Thread jsoref
Github user jsoref commented on a diff in the pull request: https://github.com/apache/cordova-docs/pull/205#discussion_r12906738 --- Diff: docs/en/edge/guide/cli/index.md --- @@ -73,22 +74,41 @@ To install the `cordova` command-line tool, follow these steps: the CLI does us

[GitHub] cordova-docs pull request: CB-6680 Clarify instructions for instal...

2014-05-21 Thread jsoref
Github user jsoref commented on a diff in the pull request: https://github.com/apache/cordova-docs/pull/205#discussion_r12906677 --- Diff: docs/en/edge/guide/cli/index.md --- @@ -73,22 +74,41 @@ To install the `cordova` command-line tool, follow these steps: the CLI does us

[GitHub] cordova-docs pull request: CB-6680 Clarify instructions for instal...

2014-05-21 Thread jsoref
Github user jsoref commented on a diff in the pull request: https://github.com/apache/cordova-docs/pull/205#discussion_r12906648 --- Diff: docs/en/edge/guide/cli/index.md --- @@ -73,22 +74,41 @@ To install the `cordova` command-line tool, follow these steps: the CLI does us

[GitHub] cordova-docs pull request: CB-6680 Clarify instructions for instal...

2014-05-21 Thread jsoref
Github user jsoref commented on a diff in the pull request: https://github.com/apache/cordova-docs/pull/205#discussion_r12906580 --- Diff: docs/en/edge/guide/cli/index.md --- @@ -73,22 +74,41 @@ To install the `cordova` command-line tool, follow these steps: the CLI does us

[GitHub] cordova-docs pull request: CB-6680 Clarify instructions for instal...

2014-05-21 Thread jsoref
Github user jsoref commented on a diff in the pull request: https://github.com/apache/cordova-docs/pull/205#discussion_r12906514 --- Diff: docs/en/edge/guide/cli/index.md --- @@ -64,8 +64,9 @@ whose assets you pull down to local working directories. To install the `cordova` com

Re: [GitHub] cordova-docs pull request: CB-6680 Clarify instructions for instal...

2014-05-19 Thread purplecabbage
Mentioning nave and nvm is out of scope IMO. If a user knows about these things, then they also already know that they can use them ... We should stay focused. Sent from my iPhone > On May 19, 2014, at 1:18 PM, cmarcelk wrote: > > Github user cmarcelk commented on the pull request: > >h

[GitHub] cordova-docs pull request: CB-6680 Clarify instructions for instal...

2014-05-19 Thread cmarcelk
Github user cmarcelk commented on the pull request: https://github.com/apache/cordova-docs/pull/205#issuecomment-43551933 Good point. I updated it to mention nave. --- 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 p

[GitHub] cordova-docs pull request: CB-6680 Clarify instructions for instal...

2014-05-19 Thread brianleroux
Github user brianleroux commented on a diff in the pull request: https://github.com/apache/cordova-docs/pull/205#discussion_r12813055 --- Diff: docs/en/edge/guide/cli/index.md --- @@ -73,22 +74,38 @@ To install the `cordova` command-line tool, follow these steps: the CLI do

[GitHub] cordova-docs pull request: CB-6680 Clarify instructions for instal...

2014-05-19 Thread cmarcelk
GitHub user cmarcelk opened a pull request: https://github.com/apache/cordova-docs/pull/205 CB-6680 Clarify instructions for installing via npm Here's my proposal regarding sudo. You can merge this pull request into a Git repository by running: $ git pull https://github.com/cma