Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated README to install node dependencies properly #5

Merged
merged 1 commit into from
Aug 27, 2012

Conversation

zzen
Copy link
Contributor

@zzen zzen commented Aug 27, 2012

Previously the README suggested installing jasmine globally (frowned upon) and didn't cover installing dustjs-linkedin at all (which of course would in turn break tests). This is what package.json is for - no need to list the dependencies again in the README. Runing npm install in the project directory takes care of all things

Previously the README suggested installing jasmine globally (frowned upon) and didn't cover installing dustjs-linkedin at all (which of course would in turn break tests). This is what package.json is for - no need to list the dependencies again in the README. Runing npm install in the project directory takes care of all things
@travisbot
Copy link

This pull request passes (merged f16cbc7 into 47d24a7).

vybs added a commit that referenced this pull request Aug 27, 2012
Updated README to install node dependencies properly
@vybs vybs merged commit 6f78c50 into LinkedInAttic:master Aug 27, 2012
@vybs
Copy link
Contributor

vybs commented Aug 27, 2012

thanks @zzen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants