From 6050d72a3a0423fd3383ab978f8c3843f91c1002 Mon Sep 17 00:00:00 2001 From: XhmikosR Date: Tue, 28 Nov 2017 10:43:09 +0200 Subject: [PATCH] Combine path and remove variable used in one place. (#24873) --- build/change-version.js | 4 ++-- build/generate-sri.js | 5 ++--- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/build/change-version.js b/build/change-version.js index f84d4e32c15d..ab4579072b2e 100755 --- a/build/change-version.js +++ b/build/change-version.js @@ -14,8 +14,8 @@ const fs = require('fs') const path = require('path') const sh = require('shelljs') + sh.config.fatal = true -const sed = sh.sed // Blame TC39... https://github.com/benjamingr/RegExp.escape/issues/37 RegExp.quote = (string) => string.replace(/[-\\^$*+?.()|[\]{}]/g, '\\$&') @@ -58,7 +58,7 @@ function replaceRecursively(directory, excludedDirectories, allowedExtensions, o replacement = RegExp.quoteReplacement(replacement) const updateFile = !DRY_RUN ? (filepath) => { if (allowedExtensions.has(path.parse(filepath).ext)) { - sed('-i', original, replacement, filepath) + sh.sed('-i', original, replacement, filepath) } } : (filepath) => { if (allowedExtensions.has(path.parse(filepath).ext)) { diff --git a/build/generate-sri.js b/build/generate-sri.js index b21311b3b12c..708e6dfbf823 100644 --- a/build/generate-sri.js +++ b/build/generate-sri.js @@ -16,11 +16,10 @@ const fs = require('fs') const path = require('path') const sriToolbox = require('sri-toolbox') const sh = require('shelljs') -const sed = sh.sed sh.config.fatal = true -const configFile = path.join(__dirname, '..', '_config.yml') +const configFile = path.join(__dirname, '../_config.yml') // Array of objects which holds the files to generate SRI hashes for. // `file` is the path from the root folder @@ -56,6 +55,6 @@ files.forEach((file) => { console.log(`${file.configPropertyName}: ${integrity}`) - sed('-i', new RegExp(`(\\s${file.configPropertyName}:\\s+"|')(\\S+)("|')`), '$1' + integrity + '$3', configFile) + sh.sed('-i', new RegExp(`(\\s${file.configPropertyName}:\\s+"|')(\\S+)("|')`), '$1' + integrity + '$3', configFile) }) })