Skip to content

Commit

Permalink
Tweak syntax in rollup build script (#9852)
Browse files Browse the repository at this point in the history
* 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:**
#9398
  • Loading branch information
flarnie committed Jun 6, 2017
1 parent 467dda6 commit 30e6c6c
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions scripts/rollup/build.js
Expand Up @@ -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: {
Expand Down

0 comments on commit 30e6c6c

Please sign in to comment.