Added npm install requirement to the README #341
Conversation
@@ -172,6 +172,12 @@ Then fill in the generated file with logic, fixtures, etc. You can then apply th | |||
|
|||
Running the Development Server | |||
------------------------------ | |||
You will need to have the Node.js package manager npm installed. You can install it by running: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This isn't a totally accurate description. We're not installing the Node.js package manager (npm), we are in fact using npm to install other dependencies that are required for One and Done.
Lots of commits here - possibly needs a rebase @rbillings? |
6b8d978
to
be64e4e
Compare
Rebased and ready for a review, @mozilla/web-qa-sorcerers ? |
Still that big user.json blog, @rbillings - https://github.com/mozilla/oneanddone/pull/341/files#diff-f3e194e100c9d799628b115d21a2e169 |
@stephendonner - thanks for looking at this. I just added comments to the README regarding the npm install, I'm unclear what you are referencing for the user.json file? |
@rbillings the pull request includes a user.json file, which I suspect is not intended if this should be just README changes. |
Thanks for noticing that extra file that snuck in! |
No description provided.