Separate detection for npm and Yarn

[pladaria]

I found this library gives false positives when used in a lifecycle script, for example, importing is-npm in a script called by prepublishOnly returns true when executing yarn publish.

This worked for me:

const isNpm = process.env.npm_execpath.includes('npm-cli.js');

[sindresorhus]

From the package description:

Check if your code is running as an npm or yarn script

[pladaria]

Sorry, I thought this package had a different purpose...

[sindresorhus]

I'm curious about your use-case. Why do you need to explicitly know whether it's npm or Yarn?

[pladaria]

I'm using yarn in my project, with a .yarnrc configuration (ignored by npm) so the results of executing npm publish and yarn publish are different.

In a prepublishOnly script I'm performing some checks, one of them is to detect if the command was executed with npm and throw an error in that case.

[sindresorhus]

Hmm, I think we could maybe do a breaking release and make it:

-const {isNpm} = require('is-npm');
+const {isNpmOrYarn, isNpm, isYarn} = require('is-npm');

That would also make it more explicit.

[pladaria]

Nice, thanks