Account for io being async

[gtanner]

If there are a lot of results, there is a chance of exiting this process before it is finished writing to the console.

Wait for stdout to be finished before exiting.

[lijunle]

Is this serious? There is only one call for console.log or console.error. I think these two operations are synchronous and block the IO. Then, is it no need to wait the finish?

[gtanner]

Yeah ;) console.error and console.warn are sync, but console.log is async, thus needing to wait for stdout to flush.

I was getting half generated JSON output on some larger objects

[gtanner]

It all depends where the "output" is going: https://nodejs.org/dist/latest-v4.x/docs/api/console.html#console_asynchronous_vs_synchronous_consoles

[gtanner]

But in the case of someone using child_process.exec to run this, the output is always a stream, which forces stdout to be async.

[codecov-io]

Current coverage is 98.10%

Merging #98 into master will not affect coverage as of 0843c73

@@            master    #98   diff @@
=====================================
  Files           16     16       
  Stmts          369    369       
  Branches        84     84       
  Methods          0      0       
=====================================
  Hit            362    362       
  Partial          5      5       
  Missed           2      2       

Review entire Coverage Diff as of 0843c73

Powered by Codecov. Updated on successful CI builds.

[lijunle]

Great information! Merged! Thanks very much! 🍻