From 9b3501310805fa2c26efc8ed1518b750e3791626 Mon Sep 17 00:00:00 2001 From: Alex Potsides Date: Tue, 24 Oct 2023 07:09:46 +0100 Subject: [PATCH] fix: re-enable test coverage (#1195) c8 has been released recently as has node so re-enable test coverage --- src/cmds/test.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/cmds/test.js b/src/cmds/test.js index 3106ab2db..d882655ad 100644 --- a/src/cmds/test.js +++ b/src/cmds/test.js @@ -107,9 +107,9 @@ export default { * @param {any} argv */ async handler (argv) { - // temporarily disable code coverage on node 18 - if (argv.cov && process.version.startsWith('v18.')) { - console.warn(kleur.red('!!! Temporarily disabling code coverage\n')) // eslint-disable-line no-console + // temporarily disable code coverage on node targets running node 18 + if (argv.cov && process.version.startsWith('v18.') && argv.target === 'node') { + console.warn(kleur.red('!!! Temporarily disabling code coverage - https://github.com/ipfs/aegir/issues/1206\n')) // eslint-disable-line no-console delete argv.cov }