[FEATURE] Prefer local over global CLI version

[matz3]

When running the global "ui5" command while a local @ui5/cli version is
installed for the current project, the local one should be preferred.

Fixes: #58

[CLAassistant]

All committers have signed the CLA.

[CLAassistant]

Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[coveralls]

Pull Request Test Coverage Report for Build 310

• 12 of 19 (63.16%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.07%) to 41.245%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
bin/ui5.js	12	19	63.16%

Totals
Change from base Build 300:	0.07%
Covered Lines:	79
Relevant Lines:	185

---

💛 - Coveralls

[RandomByte]

rebased

[RandomByte]

Is "Prefer local over global CLI version" really a fix or rather a feature? I would opt for the latter

[matz3]

Yes, I think feature is a better fit.

[RandomByte]

Ok from my side.

@matz3 please confirm that the change in commit bc59b85 is fine for you

[matz3]

@RandomByte good idea. This also helps to see whether a locally linked ui5-cli project is used.
I also like the README explanation 👍

But the ENV var is strange. I would expect to have a general "UI5 CLI" prefix that could be used in future for more variables, like UI5CLI_* or UI5_CLI_*.
And a shorter one... what about UI5_CLI_NO_LOCAL=true?
This would also fit the a command option that can be used in "XO" for the same matter, which is something to think about:
https://github.com/xojs/xo/blob/d1fd3e127b191b701716d6f775b6f5c8c5db3778/cli.js#L10

About the console.info logs. I think we shouldn't always log this. Especially not in such a verbose way. XO and AVA just use the debug module. I would only expect them with --verbose or DEBUG=*.

[RandomByte]

Done

[matz3]

Nice 👍 Seems that you need to approve as I'm the author.

[RandomByte]

Forgot to update the README with the new environment variable name. Seems to be all good now. Yours to merge.

[RandomByte]

Note (just came to my mind): When using a CLI release containing this change in a repository that uses a release without it, no INFO: Using local @ui5/cli installation information will be logged due to the missing setTimeout in older bin/ui5.js versions.

[matz3]

Yes, that's correct, but I think this can be neglected.