Skip to content
Snippets Groups Projects
Commit b9f783b5 authored by Rhys Arkins's avatar Rhys Arkins
Browse files

fix: ignoreScripts default (#18403)

Changes ignoreScripts defaulit to be `true`.

Closes #18173

BREAKING CHANGE: ignoreScripts now default to true. If allowScripts=true in global config, ignoreScripts must be set to false in repo config if you want all repos to run scripts.
parent cbca54d0
No related branches found
No related tags found
No related merge requests found
...@@ -625,9 +625,9 @@ const options: RenovateOptions[] = [ ...@@ -625,9 +625,9 @@ const options: RenovateOptions[] = [
{ {
name: 'ignoreScripts', name: 'ignoreScripts',
description: description:
'Set this to `true` if `allowScripts=true` but you wish to skip running scripts when updating lock files.', 'Set this to `false` if `allowScripts=true` and you wish to run scripts when updating lock files.',
type: 'boolean', type: 'boolean',
default: false, default: true,
supportedManagers: ['npm', 'composer'], supportedManagers: ['npm', 'composer'],
}, },
{ {
......
...@@ -79,7 +79,7 @@ export async function generateLockFiles( ...@@ -79,7 +79,7 @@ export async function generateLockFiles(
return { error: false }; return { error: false };
} }
let lernaCommand = `lerna bootstrap --no-ci --ignore-scripts -- `; let lernaCommand = `lerna bootstrap --no-ci --ignore-scripts -- `;
if (GlobalConfig.get('allowScripts') && config.ignoreScripts !== false) { if (GlobalConfig.get('allowScripts') && !config.ignoreScripts) {
cmdOptions = cmdOptions.replace('--ignore-scripts ', ''); cmdOptions = cmdOptions.replace('--ignore-scripts ', '');
lernaCommand = lernaCommand.replace('--ignore-scripts ', ''); lernaCommand = lernaCommand.replace('--ignore-scripts ', '');
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment