From 7c52b802b4cdebd369c49bb5647ba7d3f46da0a1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Paul=20O=E2=80=99Shannessy?= Date: Fri, 22 Jan 2016 10:17:30 -0800 Subject: [PATCH] Clean up ununsed devDeps Also removed object-assign. We only support building with node v4+ which has Object.assign. --- Gruntfile.js | 3 +-- grunt/tasks/jest.js | 5 ++--- grunt/tasks/npm-react-addons.js | 3 +-- package.json | 5 ----- scripts/jest/preprocessor.js | 7 ++++--- 5 files changed, 8 insertions(+), 15 deletions(-) diff --git a/Gruntfile.js b/Gruntfile.js index df98b515f3..3e5d085797 100644 --- a/Gruntfile.js +++ b/Gruntfile.js @@ -1,6 +1,5 @@ 'use strict'; -var assign = require('object-assign'); var path = require('path'); var process = require('process'); @@ -35,7 +34,7 @@ module.exports = function(grunt) { // but if it breaks we'll fix it then. cmd: path.join('node_modules', '.bin', GULP_EXE), args: args, - opts: assign({stdio: 'inherit'}, opts), + opts: Object.assign({stdio: 'inherit'}, opts), }, function(err, result, code) { if (err) { grunt.fail.fatal('Something went wrong running gulp: ', result); diff --git a/grunt/tasks/jest.js b/grunt/tasks/jest.js index 5649517d97..da406e9111 100644 --- a/grunt/tasks/jest.js +++ b/grunt/tasks/jest.js @@ -6,7 +6,6 @@ 'use strict'; -var assign = require('object-assign'); var async = require('async'); var fs = require('fs'); var glob = require('glob'); @@ -34,7 +33,7 @@ function getCollectCoverageOnlyFrom(callback) { var result = {}; async.each(patterns, function(pattern) { - var options = assign({ nodir: true }, collectCoverageOnlyFrom[pattern]); + var options = Object.assign({ nodir: true }, collectCoverageOnlyFrom[pattern]); glob(pattern, options, function(err, files) { (files || []).reduce(function(object, key) { object[key] = true; @@ -51,7 +50,7 @@ function getCollectCoverageOnlyFrom(callback) { function getJestConfig(callback) { var rootDir = path.resolve(buildPath, path.resolve(config.rootDir)); getCollectCoverageOnlyFrom(function(err, data) { - callback(err, assign({}, config, { + callback(err, Object.assign({}, config, { rootDir: rootDir, collectCoverage: true, collectCoverageOnlyFrom: data, diff --git a/grunt/tasks/npm-react-addons.js b/grunt/tasks/npm-react-addons.js index 238fefc60b..948acebf1c 100644 --- a/grunt/tasks/npm-react-addons.js +++ b/grunt/tasks/npm-react-addons.js @@ -1,6 +1,5 @@ 'use strict'; -var assign = require('object-assign'); var fs = require('fs'); var grunt = require('grunt'); var path = require('path'); @@ -76,7 +75,7 @@ function buildReleases() { var destLicense = path.join(destDir, 'LICENSE'); var destPatents = path.join(destDir, 'PATENTS'); - var pkgData = assign({}, pkgTemplate); + var pkgData = Object.assign({}, pkgTemplate); pkgData.name = pkgName; grunt.file.mkdir(destDir); diff --git a/package.json b/package.json index 76635fe84b..86eceae4d9 100644 --- a/package.json +++ b/package.json @@ -6,7 +6,6 @@ "async": "^1.5.0", "babel": "^5.8.29", "babel-eslint": "4.1.5", - "benchmark": "^1.0.0", "browserify": "^12.0.1", "bundle-collapser": "^1.1.1", "coffee-script": "^1.8.0", @@ -28,12 +27,8 @@ "gulp": "^3.9.0", "gulp-babel": "^5.3.0", "gulp-flatten": "^0.2.0", - "gulp-util": "^3.0.7", "gzip-js": "~0.3.2", "jest-cli": "^0.6.1", - "jstransform": "^11.0.0", - "object-assign": "^4.0.1", - "optimist": "^0.6.1", "platform": "^1.1.0", "run-sequence": "^1.1.4", "through2": "^2.0.0", diff --git a/scripts/jest/preprocessor.js b/scripts/jest/preprocessor.js index 3030049ee5..14739114c4 100644 --- a/scripts/jest/preprocessor.js +++ b/scripts/jest/preprocessor.js @@ -6,7 +6,6 @@ process.env.NODE_ENV = 'test'; var path = require('path'); -var assign = require('object-assign'); var babel = require('babel'); var coffee = require('coffee-script'); @@ -52,8 +51,10 @@ module.exports = { !filePath.match(/\/node_modules\//) && !filePath.match(/\/third_party\//) ) { - var rv = - babel.transform(src, assign({filename: filePath}, babelOptions)).code; + var rv = babel.transform( + src, + Object.assign({filename: filePath}, babelOptions) + ).code; // hax to turn fbjs/lib/foo into /path/to/node_modules/fbjs/lib/foo // because jest is slooow with node_modules paths (facebook/jest#465) rv = rv.replace(