@ckeditor/ckeditor5-dev-release-tools 38.3.1 → 38.4.1

package/README.md

@@ -2,7 +2,7 @@ CKEditor 5 release tools
 ========================
 
 [![npm version](https://badge.fury.io/js/%40ckeditor%2Fckeditor5-dev-release-tools.svg)](https://www.npmjs.com/package/@ckeditor/ckeditor5-dev-release-tools)
-[![Build Status](https://travis-ci.com/ckeditor/ckeditor5-dev.svg?branch=master)](https://app.travis-ci.com/github/ckeditor/ckeditor5-dev)
+[![CircleCI](https://circleci.com/gh/ckeditor/ckeditor5-dev.svg?style=shield)](https://app.circleci.com/pipelines/github/ckeditor/ckeditor5-dev?branch=master)
 
 Tasks used during a release of [CKEditor 5](https://ckeditor.com) and related packages.
 

package/lib/tasks/reassignnpmtags.js

@@ -12,10 +12,13 @@
 const chalk = require( 'chalk' );
 const columns = require( 'cli-columns' );
 const { tools } = require( '@ckeditor/ckeditor5-dev-utils' );
+const util = require( 'util' );
+const exec = util.promisify( require( 'child_process' ).exec );
 const assertNpmAuthorization = require( '../utils/assertnpmauthorization' );
 
 /**
  * Used to switch the tags from `staging` to `latest` for specified array of packages.
+ * Each operation will be retried up to 3 times in case of failure.
  *
  * @param {Object} options
  * @param {String} options.npmOwner User that is authorized to release packages.
@@ -33,30 +36,29 @@ module.exports = async function reassignNpmTags( { npmOwner, version, packages }
 	const counter = tools.createSpinner( 'Reassigning npm tags...', { total: packages.length } );
 	counter.start();
 
-	for ( const packageName of packages ) {
-		const latestVersion = await exec( `npm show ${ packageName }@latest version` )
-			.then( version => version.trim() )
-			.catch( error => {
-				errors.push( trimErrorMessage( error.message ) );
-			} );
-
-		if ( latestVersion === version ) {
-			packagesSkipped.push( packageName );
-
-			continue;
-		}
-
-		await exec( `npm dist-tag add ${ packageName }@${ version } latest` )
-			.then( () => {
-				packagesUpdated.push( packageName );
+	const updateTagPromises = packages.map( async packageName => {
+		const updateLatestTagRetryable = retry( () => exec( `npm dist-tag add ${ packageName }@${ version } latest` ) );
+		await updateLatestTagRetryable()
+			.then( response => {
+				if ( response.stdout ) {
+					packagesUpdated.push( packageName );
+				} else if ( response.stderr ) {
+					throw new Error( response.stderr );
+				}
 			} )
 			.catch( error => {
-				errors.push( trimErrorMessage( error.message ) );
+				if ( error.message.includes( 'is already set to version' ) ) {
+					packagesSkipped.push( packageName );
+				} else {
+					errors.push( trimErrorMessage( error.message ) );
+				}
 			} )
 			.finally( () => {
 				counter.increase();
 			} );
-	}
+	} );
+
+	await Promise.allSettled( updateTagPromises );
 
 	counter.finish();
 
@@ -85,9 +87,24 @@ function trimErrorMessage( message ) {
 }
 
 /**
- * @param {String} command
- * @returns {Promise.<String>}
+ * @param {Function} callback
+ * @param {Number} times
+ * @returns {RetryCallback}
  */
-async function exec( command ) {
-	return tools.shExec( command, { verbosity: 'silent', async: true } );
+function retry( callback, times = 3 ) {
+	return ( ...args ) => Promise.resolve()
+		.then( () => callback( ...args ) )
+		.catch( err => {
+			if ( times > 0 ) {
+				return retry( callback, times - 1 )( ...args );
+			}
+
+			throw err;
+		} );
 }
+
+/**
+ * @callback RetryCallback
+ * @param {...*} args
+ * @returns {Promise}
+ */

package/package.json

@@ -1,11 +1,11 @@
 {
   "name": "@ckeditor/ckeditor5-dev-release-tools",
-  "version": "38.3.1",
+  "version": "38.4.1",
   "description": "Tools used for releasing CKEditor 5 and related packages.",
   "keywords": [],
   "main": "lib/index.js",
   "dependencies": {
-    "@ckeditor/ckeditor5-dev-utils": "^38.3.1",
+    "@ckeditor/ckeditor5-dev-utils": "^38.4.1",
     "@octokit/rest": "^19.0.0",
     "chalk": "^4.0.0",
     "cli-table": "^0.3.1",