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

Redundant 'cd jquery' command in README.md #2359

Closed
elas7 opened this Issue May 29, 2015 · 2 comments

Comments

Projects
None yet
3 participants
@elas7

elas7 commented May 29, 2015

In line 169 of README.md, the following command is suggested to start 'grunt watch'

cd jquery && grunt watch

The cd jquery part seems to be redundant, considering that the previous command in the instructions is npm install.
Also, the only other part of README.md that uses cd jquery is when explicitly instructing to enter the jquery directory after cloning the jquery repo.

@mr21

This comment has been minimized.

Show comment
Hide comment
@mr21

mr21 May 29, 2015

Contributor

+1
or maybe something like this:

$> pwd
jquery
$> grunt watch
Contributor

mr21 commented May 29, 2015

+1
or maybe something like this:

$> pwd
jquery
$> grunt watch

markelog added a commit that referenced this issue May 29, 2015

@markelog markelog closed this in 3c92770 May 29, 2015

@markelog

This comment has been minimized.

Show comment
Hide comment
@markelog

markelog May 29, 2015

Member

Thank you!

Member

markelog commented May 29, 2015

Thank you!

markelog added a commit that referenced this issue Nov 10, 2015

@cssmagic cssmagic referenced this issue May 18, 2016

Open

jQuery #5

@lock lock bot locked as resolved and limited conversation to collaborators Jun 19, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.