Node: doc: typo in api/process.md

Created on 21 Apr 2017  路  6Comments  路  Source: nodejs/node

  • Version: master, v7.x, v6.x
  • Platform: n/a
  • Subsystem: doc

In doc/api/process.md, there is a typo in the 'A note on process I/O' section where it says:

Synchronous writes avoid problems such as output written with `console.log()` or `console.write()` being unexpectedly interleaved, ...

where console.write() should presumably instead be console.error(), since console.write() does not exist and console.error() is the only other console method referred to in that section.

doc good first issue process

Most helpful comment

I can make this change.

All 6 comments

I can make this change.

I have this ready but it's blocked by another PR :-(. Once that one clears I can send this one.

@morrme what do you mean by blocked? You can have lots of PRs open at the same time.

I see that your issue12565 branch also has the commits from your doc-ci-jobs branch. The fix is to do:

git fetch --all # Make sure you have the latest changes
git checkout issue12565
git push # Make sure your remote branches are up to date
git reset --hard upstream/master # Assumes nodejs/node remote is called upstream
git cherry-pick origin/issue12565 # Assumes morrme/node remote is called origin
git push --force-with-lease

The cherry-pick will just pull the top commit from your remote branch, which is the only one you want.

Then you can raise another Pull Request with: https://github.com/nodejs/node/compare/master...morrme:issue12565

@gibfahn that's what' I meant. I am so embarrassed to have made such a mistake!

I found a similar fix but we've gotten so far with the CI jobs issue, I was worried about messing it up.

I will give it a try.

Thanks for your support, as always!

I'm so embarrassed to have made such a mistake!

Git is hard, no need to be embarassed 馃榿 .

After the cherry-pick you can do git log --graph --decorate --oneline, you should see something like this (your branch is master + the doc commit):

image

I found a similar fix but we've gotten so far with the CI jobs issue, I was worried about messing it up.

That's a very reasonable thing to worry about. The trick is to run git status before you make changes (make sure you're on the right branch), and to triple-check before your force push (git push -f).

THANK YOU @gibfahn ! 馃弳

Was this page helpful?
0 / 5 - 0 ratings