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

doc: harmonize $ node command line notation #4806

Closed
wants to merge 1 commit into from
Closed

doc: harmonize $ node command line notation #4806

wants to merge 1 commit into from

Conversation

eljefedelrodeodeljefe
Copy link
Contributor

continuation of #4733.

Harmonizes inconsistent repl and command-line notation usage e.g. node>, % node, > node
to $ node

@Qard
Copy link
Member

Qard commented Jan 21, 2016

LGTM, thanks!

@cjihrig
Copy link
Contributor

cjihrig commented Jan 21, 2016

LGTM

1 similar comment
@jasnell
Copy link
Member

jasnell commented Jan 21, 2016

LGTM

@jasnell jasnell added doc Issues and PRs related to the documentations. lts-watch-v4.x labels Jan 21, 2016
@silverwind
Copy link
Contributor

Thanks, landed in 4e1023d.

@silverwind silverwind closed this Jan 22, 2016
silverwind pushed a commit that referenced this pull request Jan 22, 2016
PR-URL: #4806
Reviewed-By: Stephan Belanger <admin@stephenbelanger.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
@eljefedelrodeodeljefe eljefedelrodeodeljefe deleted the feature/doc-harmonize-command-line-notation branch January 23, 2016 12:11
rvagg pushed a commit that referenced this pull request Jan 25, 2016
PR-URL: #4806
Reviewed-By: Stephan Belanger <admin@stephenbelanger.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
@MylesBorins
Copy link
Contributor

this commit is blocked by a number of other doc changes that must be backported first

edit: for landing on lts

MylesBorins pushed a commit that referenced this pull request Feb 22, 2016
PR-URL: #4806
Reviewed-By: Stephan Belanger <admin@stephenbelanger.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
MylesBorins pushed a commit that referenced this pull request Feb 22, 2016
PR-URL: #4806
Reviewed-By: Stephan Belanger <admin@stephenbelanger.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
MylesBorins pushed a commit that referenced this pull request Mar 2, 2016
PR-URL: #4806
Reviewed-By: Stephan Belanger <admin@stephenbelanger.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
scovetta pushed a commit to scovetta/node that referenced this pull request Apr 2, 2016
PR-URL: nodejs#4806
Reviewed-By: Stephan Belanger <admin@stephenbelanger.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants