Good First Contribution: Inconsistency between CLI and README

[evenstensberg]

Is this a bug report?

No

This is what is suggested by the terminal:

This is from the README

---

As yarn is facebook made, I'd change the README example to use yarn instead of npm

[gaearon]

The CLI has detected you already have Yarn on your system, so we suggested that. The README has least-effort instructions that should work for everybody, so we decided not to put Yarn there.

I hope this explains it!

[evenstensberg]

No worries! Got another suggestion, and that is to detect if serve is installed globally before outputting if they want to install it:

Here I've already got serve installed, so it would be nice to just tell the users to run serve instead of installing it globally first

[gaearon]

That would be great. Mind filing another issue?

Unfortunately I'm not sure if it's possible to detect it easily, but maybe we could try calling serve -v silently or something.

[evenstensberg]

Don't mind at all. I think it's doable, just map process.argv back to where the global binpath is and add the module name, later checking if the directory exists. Alternatively, adding global-prefix is also a solution