From 30e6c6c9c9f8e52776981a0e91ccfbabb95f7974 Mon Sep 17 00:00:00 2001 From: Flarnie Marchan Date: Tue, 6 Jun 2017 07:26:47 -0700 Subject: [PATCH] Tweak syntax in rollup build script (#9852) * Tweak syntax in rollup build script @bvaughn and I already discussed this. **test plan:** `yarn build` * Remove JSDoc comments **what is the change?:** removing some comments **why make this change?:** The code is basically self explanatory and these comments could get out of sync. **test plan:** Visual inspection, `yarn build` and `yarn test` **issue:** https://github.com/facebook/react/issues/9398 --- scripts/rollup/build.js | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/scripts/rollup/build.js b/scripts/rollup/build.js index fe3612e870e2..7890bd71cb99 100644 --- a/scripts/rollup/build.js +++ b/scripts/rollup/build.js @@ -194,13 +194,12 @@ function getFilename(name, hasteName, bundleType) { } } -function uglifyConfig({ - mangle, - manglePropertiesOnProd, - preserveVersionHeader, - removeComments, - headerSanityCheck, -}) { +function uglifyConfig(configs) { + var mangle = configs.mangle; + var manglePropertiesOnProd = configs.manglePropertiesOnProd; + var preserveVersionHeader = configs.preserveVersionHeader; + var removeComments = configs.removeComments; + var headerSanityCheck = configs.headerSanityCheck; return { warnings: false, compress: {