fix: Remove NPM token check from publishing script (#1712)

packages/caliper-publish/artifacts/npm-publish.sh

@@ -22,15 +22,8 @@ ARCH=`uname -m`
 
 if [[ -z "${NPM_REGISTRY}" ]]
 then
-    if [[ -z "${NPM_TOKEN}" ]]
-    then
-        echo "NPM_TOKEN must be set when publishing to the public NPM registry."
-        exit 1
-    else
-        # Set the NPM access token we will use to publish.
-        npm config set registry https://registry.npmjs.org/
-        npm config set //registry.npmjs.org/:_authToken ${NPM_TOKEN}
-    fi
+    # Set the public NPM registry as default
+    npm config set registry https://registry.npmjs.org/
 fi
 
 npm publish --access public ${NPM_REGISTRY} ${DRY_RUN} --tag ${TAG}

packages/caliper-publish/lib/npmCommand.js

@@ -25,7 +25,7 @@ module.exports.builder = yargs => {
             demand: false,
             default: '',
             type: 'string',
-            describe: 'The NPM registry address to use for publishing the packages. Defaults to the public NPM registry which requires NPM_TOKEN to be set.'
+            describe: 'The NPM registry address to use for publishing the packages. Defaults to the public NPM registry which requires trusted publishing to be configured.'
         },
         'dry-run': {
             alias: 'd',

packages/caliper-publish/publish.js

@@ -27,9 +27,9 @@ let results = yargs
     .demandCommand(1, 1, 'Please specify a command to continue')
     .wrap(null)
     .strict()
+    .version(version) // enabling the version command must come before its alias setting, otherwise -v will not work
     .alias('version', 'v')
     .alias('help', 'h')
-    .version(version)
     .describe('version', 'Show version information')
     .describe('help', 'Show usage information')
     .argv;