feat: bump deps and fix contributors issue

Author: Kent C. Dodds

package.json

@@ -31,43 +31,43 @@
  "author": "Kent C. Dodds <kent@doddsfamily.us> (http://kentcdodds.com/)",
  "license": "MIT",
  "dependencies": {
- "all-contributors-cli": "^4.5.0",
+ "all-contributors-cli": "^4.9.0",
  "arrify": "^1.0.1",
  "babel-cli": "^6.26.0",
  "babel-jest": "^21.2.0",
- "babel-macros": "^1.0.2",
+ "babel-macros": "^1.2.0",
  "babel-plugin-external-helpers": "^6.22.0",
  "babel-plugin-minify-dead-code-elimination": "^0.2.0",
- "babel-plugin-module-resolver": "^2.7.1",
+ "babel-plugin-module-resolver": "^3.0.0",
  "babel-plugin-transform-class-properties": "^6.24.1",
  "babel-plugin-transform-inline-environment-variables": "^0.2.0",
  "babel-plugin-transform-object-rest-spread": "^6.26.0",
  "babel-plugin-transform-react-jsx": "^6.24.1",
  "babel-plugin-transform-react-remove-prop-types": "^0.4.10",
- "babel-preset-env": "^1.6.0",
+ "babel-preset-env": "^1.6.1",
  "babel-preset-react": "^6.24.1",
- "concurrently": "^3.5.0",
- "cross-env": "^5.0.5",
+ "concurrently": "^3.5.1",
+ "cross-env": "^5.1.1",
  "cross-spawn": "^5.1.0",
  "doctoc": "^1.3.0",
- "eslint": "^4.8.0",
- "eslint-config-kentcdodds": "^13.0.0",
- "eslint-config-prettier": "^2.6.0",
+ "eslint": "^4.11.0",
+ "eslint-config-kentcdodds": "^13.0.1",
+ "eslint-config-prettier": "^2.8.0",
  "glob": "^7.1.2",
  "husky": "^0.14.3",
  "is-ci": "^1.0.10",
  "jest": "^21.2.1",
- "lint-staged": "^4.2.3",
+ "lint-staged": "^5.0.0",
  "lodash.camelcase": "^4.3.0",
  "lodash.has": "^4.5.2",
  "mkdirp": "^0.5.1",
- "prettier": "^1.7.4",
- "read-pkg-up": "^2.0.0",
- "resolve": "^1.4.0",
+ "prettier": "^1.8.2",
+ "read-pkg-up": "^3.0.0",
+ "resolve": "^1.5.0",
  "rimraf": "^2.6.2",
- "rollup": "^0.48.2",
+ "rollup": "^0.51.8",
  "rollup-plugin-babel": "^3.0.2",
- "rollup-plugin-commonjs": "^8.2.1",
+ "rollup-plugin-commonjs": "^8.2.6",
  "rollup-plugin-json": "^2.3.0",
  "rollup-plugin-node-builtins": "^2.1.2",
  "rollup-plugin-node-globals": "^1.1.0",

src/scripts/__tests__/contributors.js

@@ -18,7 +18,7 @@ test('calls all-contributors CLI with args', () => {
  require('../contributors')
  expect(crossSpawnSyncMock).toHaveBeenCalledTimes(1)
  expect(crossSpawnSyncMock).toHaveBeenCalledWith(
- expect.stringMatching('all-contributors-cli'),
+ expect.stringMatching('all-contributors'),
  ['add'],
  {stdio: 'inherit'},
  )

src/scripts/contributors.js

@@ -1,9 +1,14 @@
 const spawn = require('cross-spawn')
+const {resolveBin} = require('../utils')
 
 const args = process.argv.slice(2)
 
-const result = spawn.sync(require.resolve('all-contributors-cli/cli'), args, {
- stdio: 'inherit',
-})
+const result = spawn.sync(
+ resolveBin('all-contributors-cli', {executable: 'all-contributors'}),
+ args,
+ {
+ stdio: 'inherit',
+ },
+)
 
 process.exit(result.status)

src/scripts/precommit/lint-staged.js

@@ -11,6 +11,7 @@ const lintStagedPath = require.resolve('lint-staged')
 const cosmiconfigPath = resolve.sync('cosmiconfig', {
  basedir: path.dirname(lintStagedPath),
 })
+const realCosmicConfig = require(cosmiconfigPath)
 
 // lint-staged uses cosmiconfig to find its configuration
 // and it has no other way to provide config
@@ -19,9 +20,13 @@ const cosmiconfigPath = resolve.sync('cosmiconfig', {
 // config so folks don't have to have that in their package.json
 function fakeCosmiconfig(...args) {
  if (args[0] === 'lint-staged') {
- return Promise.resolve({config: require('../../config/lintstagedrc')})
+ return {
+ load() {
+ return Promise.resolve({config: require('../../config/lintstagedrc')})
+ },
+ }
  } else {
- return require(cosmiconfigPath)(...args)
+ return realCosmicConfig(...args)
  }
 }