Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

docs(tutorial/Tutorial): mention package.json file #12040

Closed
wants to merge 2 commits into from

Conversation

dlink
Copy link
Contributor

@dlink dlink commented Jun 6, 2015

If reader does not cd into angular-phonecat subdirectory - npm install will not work. This comment helps newbies understand npm install is dependent on a custom project.json file.

If reader does not cd into angular-phonecat subdirectory - npm install will not work.   This comment helps newbies understand npm install is dependent on a custom project.json file.
@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@dlink
Copy link
Contributor Author

dlink commented Jun 6, 2015

I signed it!

@googlebot
Copy link

CLAs look good, thanks!

@googlebot googlebot added cla: yes and removed cla: no labels Jun 6, 2015
@@ -129,7 +129,7 @@ Once you have Node.js installed on your machine you can download the tool depend
npm install
```

This command will download the following tools, into the `node_modules` directory:
This command reads angular-phonecat's project.json file and will download the following tools, into the `node_modules` directory:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume you mean package.json?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're right. sorry for the typo. I fixed it in the branch.

On Mon, Jun 8, 2015 at 2:07 PM, Martin Staffa [email protected]
wrote:

In docs/content/tutorial/index.ngdoc
#12040 (comment):

@@ -129,7 +129,7 @@ Once you have Node.js installed on your machine you can download the tool depend
npm install


-This command will download the following tools, into the `node_modules` directory:
+This command reads angular-phonecat's project.json file and will download the following tools, into the `node_modules` directory:

I assume you mean package.json?


Reply to this email directly or view it on GitHub
https://github.com/angular/angular.js/pull/12040/files#r31942048.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How do I get out of these discussions? I'm not quite sure what this all means? Thanks!

Sent from my iPhone

On Jun 8, 2015, at 2:07 PM, Martin Staffa [email protected] wrote:

In docs/content/tutorial/index.ngdoc:

@@ -129,7 +129,7 @@ Once you have Node.js installed on your machine you can download the tool depend
npm install


-This command will download the following tools, into the `node_modules` directory:
+This command reads angular-phonecat's project.json file and will download the following tools, into the `node_modules` directory:
I assume you mean package.json?


Reply to this email directly or view it on GitHub.

@Narretz Narretz added this to the Backlog milestone Jun 8, 2015
@Narretz Narretz closed this in ffac747 Jun 9, 2015
netman92 pushed a commit to netman92/angular.js that referenced this pull request Aug 8, 2015
If the reader does not cd into angular-phonecat subdirectory,
npm install will not work.
This comment helps newbies understand npm install is dependent
on a custom project.json file.

Closes angular#12040
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants