Use the --use-npm flag also for version checking

[Keraito]

Issue: While I was working on #3531, I noticed that the package manager responsible for checking the versions of dependencies in the generators was dependent on has_yarn, fully ignoring the --use-npm flag #3531 (comment).

What I did

I made the latestVersion function to adhere to the --use-npm flag.
How I currently did it is pass the npmOptions object all the way through generators in order to reach latestVersion.

• It works, but I wouldn't call this cleanest way of doing it, as it requires a change in all the generators. Is there a better way of accessing the --use-npm flag in the helper?
• It currently only works on a Create React App project, all the other generators will fail due to the destructuring in https://github.com/storybooks/storybook/compare/master...Keraito:use-npm?expand=1#diff-8d17cc91321b260aaa782e8a42cd5634R37, but I wanted to verify whether this is the best way or there is a more cleaner and less impactful way of doing it.

[Hypnosphi]

I think it's ok

[codecov]

Codecov Report

Merging #3535 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #3535      +/-   ##
==========================================
- Coverage   37.12%   37.11%   -0.01%     
==========================================
  Files         509      509              
  Lines       10848    10849       +1     
  Branches      971      973       +2     
==========================================
  Hits         4027     4027              
- Misses       6231     6238       +7     
+ Partials      590      584       -6

Impacted Files	Coverage Δ
lib/cli/bin/generate.js	0% <0%> (ø)	⬆️
lib/cli/lib/helpers.js	0% <0%> (ø)	⬆️
lib/cli/lib/latest_version.js	0% <0%> (ø)	⬆️
lib/ui/src/modules/ui/components/layout/usplit.js	38.7% <0%> (ø)	⬆️
lib/ui/src/modules/ui/libs/is_mobile_device.js	42.85% <0%> (ø)	⬆️
...i/src/modules/ui/components/stories_panel/index.js	20.63% <0%> (ø)	⬆️
addons/actions/src/lib/types/nan/index.js	82.35% <0%> (ø)	⬆️
lib/core/src/client/manager/provider.js	0% <0%> (ø)	⬆️
lib/ui/src/modules/ui/configs/init_panels.js	25% <0%> (ø)	⬆️
...ns/viewport/src/manager/components/viewportInfo.js	33.33% <0%> (ø)	⬆️
... and 64 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4f3137f...8d9fbb7. Read the comment docs.

[Keraito]

Tried to make sure that I didn't miss a generator or call to either latestversion/getVersion. I ran the tests in lib/cli and everything went well except for some things at the end, but that could also have been caused by some internet issues on my side. Will test again tomorrow.

[Keraito]

Ran the tests again and now they succeeded ( Done in 714.76s.), yesterday was very likely to be caused by network issues.

[Hypnosphi]

Please update generators for new apps (html, marko) as well

[Keraito]

Done, I'm not in a position atm to run the tests again though, I can do that in a few hours.

[Hypnosphi]

Released as 3.4.5