feat: allow to configure the registry via NPM_CONFIG_REGISTRY

Author: pvdlg

README.md

@@ -72,7 +72,9 @@ Use either `NPM_TOKEN` for token authentication or `NPM_USERNAME`, `NPM_PASSWORD
 
 The plugin uses the [`npm` CLI](https://github.com/npm/cli) which will read the configuration from [`.npmrc`](https://docs.npmjs.com/files/npmrc). See [`npm config`](https://docs.npmjs.com/misc/config) for the option list.
 
-The [`registry`](https://docs.npmjs.com/misc/registry) and [`dist-tag`](https://docs.npmjs.com/cli/dist-tag) can be configured in the `package.json` and will take precedence over the configuration in `.npmrc`:
+The [`registry`](https://docs.npmjs.com/misc/registry) can be configured via the npm environment variable `NPM_CONFIG_REGISTRY` and will take precedence over the configuration in `.npmrc`.
+
+The [`registry`](https://docs.npmjs.com/misc/registry) and [`dist-tag`](https://docs.npmjs.com/cli/dist-tag) can be configured in the `package.json` and will take precedence over the configuration in `.npmrc` and `NPM_CONFIG_REGISTRY`:
 ```json
 {
   "publishConfig": {

lib/get-registry.js

@@ -2,10 +2,10 @@ const path = require('path');
 const rc = require('rc');
 const getRegistryUrl = require('registry-auth-token/registry-url');
 
-module.exports = ({publishConfig: {registry} = {}, name}, {cwd}) =>
-  registry
-    ? registry
-    : getRegistryUrl(
-        name.split('/')[0],
-        rc('npm', {registry: 'https://registry.npmjs.org/'}, {config: path.resolve(cwd, '.npmrc')})
-      );
+module.exports = ({publishConfig: {registry} = {}, name}, {cwd, env}) =>
+  registry ||
+  env.NPM_CONFIG_REGISTRY ||
+  getRegistryUrl(
+    name.split('/')[0],
+    rc('npm', {registry: 'https://registry.npmjs.org/'}, {config: path.resolve(cwd, '.npmrc')})
+  );

test/get-registry.test.js

@@ -6,30 +6,39 @@ import getRegistry from '../lib/get-registry';
 
 test('Get default registry', t => {
   const cwd = tempy.directory();
-  t.is(getRegistry({name: 'package-name'}, {cwd}), 'https://registry.npmjs.org/');
-  t.is(getRegistry({name: 'package-name', publishConfig: {}}, {cwd}), 'https://registry.npmjs.org/');
+  t.is(getRegistry({name: 'package-name'}, {cwd, env: {}}), 'https://registry.npmjs.org/');
+  t.is(getRegistry({name: 'package-name', publishConfig: {}}, {cwd, env: {}}), 'https://registry.npmjs.org/');
 });
 
 test('Get the registry configured in ".npmrc" and normalize trailing slash', async t => {
   const cwd = tempy.directory();
   await appendFile(path.resolve(cwd, '.npmrc'), 'registry = https://custom1.registry.com');
 
-  t.is(getRegistry({name: 'package-name'}, {cwd}), 'https://custom1.registry.com/');
+  t.is(getRegistry({name: 'package-name'}, {cwd, env: {}}), 'https://custom1.registry.com/');
 });
 
 test('Get the registry configured from "publishConfig"', async t => {
   const cwd = tempy.directory();
   await appendFile(path.resolve(cwd, '.npmrc'), 'registry = https://custom2.registry.com');
 
   t.is(
-    getRegistry({name: 'package-name', publishConfig: {registry: 'https://custom3.registry.com/'}}, {cwd}),
+    getRegistry({name: 'package-name', publishConfig: {registry: 'https://custom3.registry.com/'}}, {cwd, env: {}}),
     'https://custom3.registry.com/'
   );
 });
 
+test('Get the registry configured in "NPM_CONFIG_REGISTRY"', t => {
+  const cwd = tempy.directory();
+
+  t.is(
+    getRegistry({name: 'package-name'}, {cwd, env: {NPM_CONFIG_REGISTRY: 'https://custom1.registry.com/'}}),
+    'https://custom1.registry.com/'
+  );
+});
+
 test('Get the registry configured in ".npmrc" for scoped package', async t => {
   const cwd = tempy.directory();
   await appendFile(path.resolve(cwd, '.npmrc'), '@scope:registry = https://custom3.registry.com');
 
-  t.is(getRegistry({name: '@scope/package-name'}, {cwd}), 'https://custom3.registry.com/');
+  t.is(getRegistry({name: '@scope/package-name'}, {cwd, env: {}}), 'https://custom3.registry.com/');
 });