Skip to content

Commit c74f81e

Browse files
committed
NPM: Use the package name to look up the npm owners
Follow-up to 8a71c2e which uses the Release.project property. For QUnit, the npm package name is 'qunitjs', not 'qunit'.
1 parent 82df2ce commit c74f81e

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

lib/npm.js

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@ module.exports = function( Release ) {
1414
return user.output.trim();
1515
},
1616

17-
_getNpmOwners: function() {
18-
var owners = Release.exec( "npm owner ls " + Release.project, { silent: true } );
17+
_getNpmOwners: function( npmPackage ) {
18+
var owners = Release.exec( "npm owner ls " + npmPackage, { silent: true } );
1919

2020
if ( owners.code !== 0 ) {
2121
Release.abort( "Error getting npm owners." );
@@ -31,11 +31,12 @@ module.exports = function( Release ) {
3131
return;
3232
}
3333

34-
var user = Release._getNpmUser(),
35-
owners = Release._getNpmOwners();
34+
var npmPackage = Release.readPackage().name,
35+
user = Release._getNpmUser(),
36+
owners = Release._getNpmOwners( npmPackage );
3637

3738
if ( owners.indexOf( user ) === -1 ) {
38-
Release.abort( user + " is not an owner of " + Release.project + " on npm." );
39+
Release.abort( user + " is not an owner of " + npmPackage + " on npm." );
3940
}
4041
},
4142

0 commit comments

Comments
 (0)