search

springernature / eslint-config-springernature

ESLint shareable config used at Springer Nature
MIT License
3 stars 0 forks source link

Add more Yarn installation info #11

Closed sonniesedge closed 6 years ago

sonniesedge commented 6 years ago

Add a quick addition based upon installing this package on a real Yarn environment.

sonniesedge commented 6 years ago

I'd advocate adding it as one of many options. We're already showing a install-peerdeps option in https://github.com/springernature/eslint-config-springernature/blame/master/README.md#L52

I agree that we shouldn't force a global install on users of the package - that's very naughty. But showing an optional quick/automated installation method using a global package - I can't see a fundamental problem with that.

I'll take a look and see if there's also a npx-ish way of doing things with Yarn.

sonniesedge commented 6 years ago

Hmmm, interestingly, there's no direct equivalent of npx in Yarn. The nearest is the create command, which isn't that useful here I think.

josebolos commented 6 years ago

How about just using npx? From what I can get that npx does, I don't think that using yarn to install deps prevents us from using npx.

From your link:

npx performs no operations which clash with people using other package managers. It doesn't even require npm to be on the system, so you can npm rm -g npm and npx will work just fine.

sonniesedge commented 6 years ago

Back from lunch. Hallo! I agree with that npx could be easily used in this case. 👍 I've updated the PR to reflect that. What do you think?

sonniesedge commented 6 years ago

Hurrah!