From 925ace479a3c64c2e047481e94f9177bda65e062 Mon Sep 17 00:00:00 2001 From: Sun Yimin Date: Fri, 21 Apr 2023 15:27:32 +0800 Subject: [PATCH 1/2] Ignore pnpm test for node14 according error message of the actions and https://github.com/pnpm/pnpm/issues/6297 --- test/utils.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/utils.js b/test/utils.js index 969744720..5f703390e 100644 --- a/test/utils.js +++ b/test/utils.js @@ -167,8 +167,8 @@ module.exports.filesAfter = function (b, n) { }; module.exports.shouldSkipPnpm = function () { - const REQUIRED_MAJOR_VERSION = 14; - const REQUIRED_MINOR_VERSION = 19; + const REQUIRED_MAJOR_VERSION = 16; + const REQUIRED_MINOR_VERSION = 14; const MAJOR_VERSION = parseInt(process.version.match(/v([0-9]+)/)[1], 10); const MINOR_VERSION = parseInt( From 5b387ff73e51df330a47d676957dacf777020440 Mon Sep 17 00:00:00 2001 From: Sun Yimin Date: Fri, 21 Apr 2023 15:43:49 +0800 Subject: [PATCH 2/2] trigger ci --- test/utils.js | 1 + 1 file changed, 1 insertion(+) diff --git a/test/utils.js b/test/utils.js index 5f703390e..f0181e870 100644 --- a/test/utils.js +++ b/test/utils.js @@ -167,6 +167,7 @@ module.exports.filesAfter = function (b, n) { }; module.exports.shouldSkipPnpm = function () { + // pnpm 8 requires at least Node.js v16.14 const REQUIRED_MAJOR_VERSION = 16; const REQUIRED_MINOR_VERSION = 14;