From f1a480bc03281340d59a636b91cdc3e48c2a2ca5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C3=ABl=20De=20Boey?= Date: Tue, 22 Nov 2022 15:12:44 +0100 Subject: [PATCH] chore: fix remarks --- index.js | 39 +++++++++++++++++++++------------------ 1 file changed, 21 insertions(+), 18 deletions(-) diff --git a/index.js b/index.js index e0153a37..abbab5f3 100755 --- a/index.js +++ b/index.js @@ -148,27 +148,31 @@ const hasDevDependency = (dependency, packageJson) => { ) } -const runSValues = [new RegExp('run-s')] +const runSValues = [ + /npm-run-all .* -s(?:$|\s|&|>|<)/, + /npm-run-all .* --sequential/, + /npm-run-all .* --serial/, + /run-s/, +] const hasRunS = (packageJson) => { - if (hasDevDependency('npm-run-all', packageJson)) { - const scripts = packageJson.scripts - const betterScripts = packageJson.betterScripts - - if (scripts) { - return Object.values(scripts).some((script) => - runSValues.some((runSValue) => runSValue.test(script)), - ) - } + if (!hasDevDependency('npm-run-all', packageJson)) { + return false + } - if (betterScripts) { - return Object.values(betterScripts).some((script) => - runSValues.some((runSValue) => runSValue.test(script)), - ) - } + const scripts = packageJson.scripts + const betterScripts = packageJson.betterScripts + if (scripts) { + return Object.values(scripts).some((script) => + runSValues.some((runSValue) => runSValue.test(script)), + ) } - return false + if (betterScripts) { + return Object.values(betterScripts).some((script) => + runSValues.some((runSValue) => runSValue.test(script)), + ) + } } const sortScripts = onObject((scripts, packageJson) => { @@ -195,9 +199,8 @@ const sortScripts = onObject((scripts, packageJson) => { ), [], ) - const toReturn = sortObjectKeys(scripts, order) - return toReturn + return sortObjectKeys(scripts, order) }) // fields marked `vscode` are for `Visual Studio Code extension manifest` only