Fix checking collaborators when publishing to external registry (#491)

sindresorhus · Jan 25, 2020 · 4ad4146 · 4ad4146
1 parent b14dbc9
commit 4ad4146
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/source/npm/util.js b/source/npm/util.js
@@ -39,11 +39,17 @@ exports.username = async ({externalRegistry}) => {
 	}
 };
 
-exports.collaborators = async packageName => {
+exports.collaborators = async pkg => {
+	const packageName = pkg.name;
 	ow(packageName, ow.string);
 
+	const args = ['access', 'ls-collaborators', packageName];
+	if (exports.isExternalRegistry(pkg)) {
+		args.push('--registry', pkg.publishConfig.registry);
+	}
+
 	try {
-		const {stdout} = await execa('npm', ['access', 'ls-collaborators', packageName]);
+		const {stdout} = await execa('npm', args);
 		return stdout;
 	} catch (error) {
 		// Ignore non-existing package error

diff --git a/source/prerequisite-tasks.js b/source/prerequisite-tasks.js
@@ -36,7 +36,7 @@ module.exports = (input, pkg, options) => {
 					externalRegistry: isExternalRegistry ? pkg.publishConfig.registry : false
 				});
 
-				const collaborators = await npm.collaborators(pkg.name);
+				const collaborators = await npm.collaborators(pkg);
 				if (!collaborators) {
 					return;
 				}