revert just-camel-case; use to-camel-case; upgrade xo

Author: heygrady

package.json

@@ -46,30 +46,31 @@
     "babel-core": "^7.0.0-bridge.0",
     "babel-jest": "^23.6.0",
     "es-check": "^4.0.0",
+    "eslint": "^5.7.0",
     "eslint-config-jest-files": "^0.1.3",
     "eslint-config-unicorn-camelcase": "^0.1.1",
     "eslint-plugin-prettier": "^3.0.0",
     "flux-standard-action": "^2.0.3",
     "husky": "^1.1.2",
     "jest": "^23.6.0",
     "lint-staged": "^7.3.0",
-    "npm-run-all": "^4.1.2",
+    "npm-run-all": "^4.1.3",
     "prettier": "^1.14.3",
     "rimraf": "^2.6.2",
-    "rollup": "^0.66.2",
-    "rollup-plugin-babel": "^4.0.1",
+    "rollup": "^0.66.6",
+    "rollup-plugin-babel": "^4.0.3",
     "rollup-plugin-commonjs": "^9.2.0",
     "rollup-plugin-node-resolve": "^3.4.0",
-    "rollup-plugin-replace": "^2.0.0",
+    "rollup-plugin-replace": "^2.1.0",
     "rollup-plugin-terser": "^3.0.0",
-    "xo": "^0.20.3"
+    "xo": "^0.23.0"
   },
   "dependencies": {
     "invariant": "^2.2.4",
-    "just-camel-case": "^3.1.0",
     "just-curry-it": "^3.1.0",
     "loose-envify": "^1.4.0",
-    "reduce-reducers": "^0.4.3"
+    "reduce-reducers": "^0.4.3",
+    "to-camel-case": "^1.0.0"
   },
   "husky": {
     "hooks": {
@@ -88,7 +89,9 @@
       "unicorn-camelcase",
       "jest-files"
     ],
-    "ignores": ["rollup.config.js"]
+    "ignores": [
+      "rollup.config.js"
+    ]
   },
   "prettier": {
     "singleQuote": true,

rollup.config.js

@@ -34,13 +34,9 @@ export default [
     input: 'src/index.js',
     output: { file: 'es/redux-actions.mjs', format: 'es', indent: false },
     plugins: [
-      nodeResolve({
-        jsnext: true
-      }),
+      nodeResolve(),
       commonjs(),
-      replace({
-        'process.env.NODE_ENV': JSON.stringify('production')
-      }),
+      replace({ 'process.env.NODE_ENV': JSON.stringify('production') }),
       terser({
         compress: {
           pure_getters: true,
@@ -62,14 +58,10 @@ export default [
       indent: false
     },
     plugins: [
-      nodeResolve({
-        jsnext: true
-      }),
+      nodeResolve(),
       commonjs(),
       babel(),
-      replace({
-        'process.env.NODE_ENV': JSON.stringify('development')
-      })
+      replace({ 'process.env.NODE_ENV': JSON.stringify('development') })
     ]
   },
 
@@ -83,14 +75,10 @@ export default [
       indent: false
     },
     plugins: [
-      nodeResolve({
-        jsnext: true
-      }),
+      nodeResolve(),
       commonjs(),
       babel(),
-      replace({
-        'process.env.NODE_ENV': JSON.stringify('production')
-      }),
+      replace({ 'process.env.NODE_ENV': JSON.stringify('production') }),
       terser({
         compress: {
           pure_getters: true,

src/utils/camelCase.js

@@ -1,6 +1,4 @@
-import justCamelCase from 'just-camel-case';
-
-const camelCase = string => justCamelCase(string, { strict: true });
+import camelCase from 'to-camel-case';
 
 const namespacer = '/';
 

src/utils/ownKeys.js

@@ -2,6 +2,13 @@ import isMap from './isMap';
 
 export default function ownKeys(object) {
   if (isMap(object)) {
+    // We are using loose transforms in babel. Here we are trying to convert an
+    // interable to an array. Loose mode expects everything to already be an
+    // array. The problem is that our eslint rules encourage us to prefer
+    // spread over Array.from.
+    //
+    // Instead of disabling loose mode we simply disable the warning.
+    // eslint-disable-next-line unicorn/prefer-spread
     return Array.from(object.keys());
   }
 

test/camelCase.test.js

@@ -9,5 +9,5 @@ test('includes forward slashes in words', () => {
 });
 
 test('does nothing to an already camel-cased action type', () => {
-  expect(camelCase('myAction')).toBe('myaction');
+  expect(camelCase('myAction')).toBe('myAction');
 });