From 9c7600d6d34c3d7d897b7a7320d2953bac52b7ed Mon Sep 17 00:00:00 2001 From: Rhys Arkins <rhys@arkins.net> Date: Wed, 9 May 2018 14:20:10 +0200 Subject: [PATCH] fix: check npmrc before calling includes --- lib/manager/npm/post-update/index.js | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/lib/manager/npm/post-update/index.js b/lib/manager/npm/post-update/index.js index d2d409c25a..fc205791f0 100644 --- a/lib/manager/npm/post-update/index.js +++ b/lib/manager/npm/post-update/index.js @@ -170,10 +170,12 @@ async function writeExistingFiles(config, packageFiles) { } } const npmrc = packageFile.npmrc || config.npmrc; - if (npmrc.includes('=${' && !config.global.exposeEnv)) { - logger.info('Skipping .npmrc as it contains variables'); - } else { - await fs.outputFile(upath.join(basedir, '.npmrc'), npmrc); + if (npmrc) { + if (npmrc.includes('=${' && !config.global.exposeEnv)) { + logger.info('Skipping .npmrc as it contains variables'); + } else { + await fs.outputFile(upath.join(basedir, '.npmrc'), npmrc); + } } if (packageFile.yarnrc) { logger.debug(`Writing .yarnrc to ${basedir}`); -- GitLab