chore(deps): update all dependencies and configs

Closes #32

Author: Kent C. Dodds

.travis.yml

@@ -8,15 +8,13 @@ branches:
     - master
 notifications:
   email: false
-node_js:
-  - 4
+node_js: 6
 before_install:
   - npm i -g npm@^3.0.0
 before_script:
   - npm prune
 script:
-  - npm run cover
-  - npm run build
+  - npm run validate
 after_success:
-  - npm run semantic-release
   - npm run report-coverage
+  - npm run semantic-release

package.json

@@ -5,15 +5,13 @@
   "main": "dist/index.js",
   "scripts": {
     "commit": "git-cz",
-    "prebuild": "npm run lint && npm run clean",
-    "clean": "rimraf dist && mkdir dist",
-    "cover": "nyc --reporter lcov --reporter text npm t",
-    "check-coverage": "nyc check-coverage --statements 100 --branches 100 --functions 100 --lines 100",
+    "prebuild": "rimraf dist",
     "report-coverage": "cat ./coverage/lcov.info | codecov",
     "build": "babel src/ -d dist/",
-    "test": "ava --require babel-register",
-    "test:watch": "nodemon --exec npm t",
+    "test": "cross-env NODE_ENV=test nyc ava",
+    "test:watch": "ava --watch",
     "lint": "eslint src/",
+    "validate": "npm-run-all lint test build",
     "semantic-release": "semantic-release pre && npm publish && semantic-release post"
   },
   "repository": {
@@ -31,39 +29,72 @@
   },
   "homepage": "https://github.com/eggheadio-github/stack-overflow-copy-paste#readme",
   "devDependencies": {
-    "ava": "0.13.0",
-    "babel-cli": "6.6.5",
-    "babel-preset-es2015": "6.6.0",
-    "babel-preset-stage-2": "6.5.0",
-    "babel-register": "6.7.2",
+    "ava": "0.15.2",
+    "babel-cli": "6.11.4",
+    "babel-plugin-istanbul": "1.0.3",
+    "babel-preset-es2015": "6.9.0",
+    "babel-preset-stage-2": "6.11.0",
+    "babel-register": "6.11.6",
     "codecov.io": "0.1.6",
-    "commitizen": "2.7.2",
-    "cz-conventional-changelog": "1.1.5",
-    "eslint": "2.4.0",
-    "eslint-config-kentcdodds": "6.0.0",
-    "ghooks": "1.0.3",
-    "nodemon": "1.9.1",
-    "nyc": "6.1.1",
-    "rimraf": "2.5.2",
+    "commitizen": "2.8.2",
+    "cross-env": "2.0.0",
+    "cz-conventional-changelog": "1.1.6",
+    "eslint": "3.2.0",
+    "eslint-config-kentcdodds": "9.0.0",
+    "ghooks": "1.3.2",
+    "nodemon": "1.10.0",
+    "npm-run-all": "2.3.0",
+    "nyc": "7.1.0",
+    "rimraf": "2.5.4",
     "semantic-release": "4.3.5",
-    "validate-commit-msg": "2.4.0"
+    "validate-commit-msg": "2.6.1"
   },
   "config": {
     "ghooks": {
       "commit-msg": "validate-commit-msg",
-      "pre-commit": "npm run cover && npm run build"
+      "pre-commit": "npm run validate"
     },
     "commitizen": {
-      "path": "node_modules/cz-conventional-changelog"
+      "path": "cz-conventional-changelog"
     }
   },
+  "nyc": {
+    "all": true,
+    "check-coverage": true,
+    "branches": 100,
+    "function": 100,
+    "lines": 100,
+    "statements": 100,
+    "reporter": [
+      "text",
+      "lcov"
+    ],
+    "include": [
+      "src"
+    ],
+    "sourceMap": false,
+    "instrument": false,
+    "require": [
+      "babel-register"
+    ]
+  },
   "babel": {
     "presets": [
       "es2015",
       "stage-2"
-    ]
+    ],
+    "env": {
+      "test": {
+        "plugins": [
+          "istanbul"
+        ]
+      }
+    }
   },
   "eslintConfig": {
-    "extends": "kentcdodds"
+    "extends": [
+      "kentcdodds",
+      "kentcdodds/ava"
+    ]
   }
 }

src/array-fill.js

@@ -1,3 +1,4 @@
+/* eslint prefer-spread:0 */
 export default fill
 
 /**
@@ -12,4 +13,3 @@ export default fill
 function fill(array, value) {
   return Array.apply(null, array).map(value.constructor.prototype.valueOf, value)
 }
-

src/flatten.js

@@ -8,8 +8,8 @@ export default flatten
  * @param {...Array} arrays - The array(s) to flatten
  * @return {Array} - The flattened array
  */
-function flatten() {
-  return [].slice.call(arguments).reduce(function flattenReducer(flat, toFlatten) {
-    return flat.concat(Array.isArray(toFlatten) ? flatten.apply(null, toFlatten) : toFlatten)
+function flatten(...args) {
+  return args.reduce(function flattenReducer(flat, toFlatten) {
+    return flat.concat(Array.isArray(toFlatten) ? flatten(...toFlatten) : toFlatten)
   }, [])
 }

src/get-query-string-param.js

@@ -11,8 +11,7 @@ export default getQueryStringParam
  */
 function getQueryStringParam(url, name) {
   const regexReadyName = name.replace(/[\[]/, '\\[').replace(/[\]]/, '\\]')
-  const regex = new RegExp('[\\?&]' + regexReadyName + '=([^&#]*)')
+  const regex = new RegExp(`[\\?&]${regexReadyName}=([^&#]*)`)
   const results = regex.exec(url)
   return results === null ? '' : decodeURIComponent(results[1].replace(/\+/g, ' '))
 }
-

test/array-fill.js

@@ -5,21 +5,21 @@ test('fills an array with a number', t => {
   const original = [1, 2, 3, 4]
   const expected = [7, 7, 7, 7]
   const actual = arrayFill(original, 7)
-  t.same(actual, expected)
+  t.deepEqual(actual, expected)
 })
 
 test('fills an array with a string', t => {
   const original = Array(4)
   const expected = ['wookie', 'wookie', 'wookie', 'wookie']
   const actual = arrayFill(original, 'wookie')
-  t.same(actual, expected)
+  t.deepEqual(actual, expected)
 })
 
 test('fills an array with a boolean', t => {
   const original = Array(4)
   const expected = [false, false, false, false]
   const actual = arrayFill(original, false)
-  t.same(actual, expected)
+  t.deepEqual(actual, expected)
 })
 
 test.todo('allow for non-primitive values like objects, arrays, and dates')

test/flatten.test.js

@@ -5,7 +5,7 @@ test('flattens an array of arrays', t => {
   const original = [[1, 2], 3, [4, 5]]
   const expected = [1, 2, 3, 4, 5]
   const actual = flatten(original)
-  t.same(actual, expected)
+  t.deepEqual(actual, expected)
 })
 
 test('flattens multiple arrays', t => {
@@ -14,5 +14,5 @@ test('flattens multiple arrays', t => {
   const original3 = [true, false, true, false]
   const expected = [1, 2, 3, 4, 5, 'A', 'b', 'C', 'd', 'E', true, false, true, false]
   const actual = flatten(original1, original2, original3)
-  t.same(actual, expected)
+  t.deepEqual(actual, expected)
 })

test/get-query-string-param.test.js

@@ -7,12 +7,12 @@ test('gets the given parameter', t => {
   const name = 'utm_campaign'
   const expected = 'contributing-to-open-source'
   const actual = getQueryStringParam(url, name)
-  t.same(actual, expected)
+  t.deepEqual(actual, expected)
 })
 
 test('returns an empty string if param does not exist', t => {
   const name = 'missing-thing'
   const expected = ''
   const actual = getQueryStringParam(url, name)
-  t.same(actual, expected)
+  t.deepEqual(actual, expected)
 })

test/pad-left.test.js

@@ -7,22 +7,22 @@ test('pads left of the given string', t => {
   const padLength = 5
   const padWith = 'z'
   const actual = padLeft(original, padLength, padWith)
-  t.same(actual, expected)
+  t.deepEqual(actual, expected)
 })
 
 test('defaults to pad a zero', t => {
   const original = '123'
   const expected = '00123'
   const padLength = 5
   const actual = padLeft(original, padLength)
-  t.same(actual, expected)
+  t.deepEqual(actual, expected)
 })
 
 test('does not pad a string longer than the pad length', t => {
   const original = '1234'
   const expected = '1234'
   const padLength = 3
   const actual = padLeft(original, padLength)
-  t.same(actual, expected)
+  t.deepEqual(actual, expected)
 })
 

test/snake-to-camel.test.js

@@ -5,12 +5,12 @@ test('converts snake-case to camelCase', t => {
   const original = 'snake-case-string'
   const expected = 'snakeCaseString'
   const actual = snakeToCamel(original)
-  t.same(actual, expected)
+  t.deepEqual(actual, expected)
 })
 
 test('converts snake_case to camelCase', t => {
   const original = 'snake_case_string'
   const expected = 'snakeCaseString'
   const actual = snakeToCamel(original)
-  t.same(actual, expected)
+  t.deepEqual(actual, expected)
 })