-
Notifications
You must be signed in to change notification settings - Fork 97
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
Download: Add a "Downloading jQuery using npm" section #123
Conversation
``` | ||
npm install jquery | ||
``` | ||
This will install jQuery to the `node_modules` directory. Within `node_modules/jquery/dist/` you will find an uncompressed release, a compressed release, and a map file. |
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.
Perhaps "to the node_modules
directory"?
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.
Isn't that what I've written?
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.
Oh dear, I meant "in the node_modules
directory"
Apart from my minor comment, the PR LGTM. |
8b34830
to
23a77ea
Compare
Fixes #122
Screenshot:
The Bower section looks really similar now; perhaps we should simplify it now that it's not a recommended way to get jQuery? We could change it to a simple mention and redirect people to Bower docs for more info (e.g. target directory). Thoughts?