[chore] Added prettier via eslint

Author: Aditya Vohra

.eslintrc.js

@@ -1,12 +1,24 @@
 module.exports = {
-  extends: 'react-app',
+  extends: ['react-app', 'prettier'],
+
+  plugins: ['prettier'],
+
+  rules: {
+    'prettier/prettier': [
+      'error',
+      {
+        semi: false,
+        singleQuote: true
+      }
+    ]
+  },
 
   overrides: [
     {
       files: 'test/**/*.js',
       env: {
-          jest: true,
-      },
-    },
-  ],
+        jest: true
+      }
+    }
+  ]
 }

build/use-lodash-es.js

@@ -1,4 +1,4 @@
-module.exports = function () {
+module.exports = function() {
   return {
     visitor: {
       ImportDeclaration(path) {

package-lock.json

@@ -15,7 +15,7 @@
   ],
   "scripts": {
     "clean": "rimraf lib dist es coverage",
-    "lint": "eslint src test build",
+    "lint": "eslint src test build --fix",
     "test": "cross-env BABEL_ENV=commonjs jest",
     "test:watch": "npm test -- --watch",
     "test:cov": "npm test -- --coverage",
@@ -99,14 +99,17 @@
     "babel-register": "^6.26.0",
     "cross-env": "^5.1.0",
     "eslint": "^4.9.0",
+    "eslint-config-prettier": "^2.6.0",
     "eslint-config-react-app": "^2.0.1",
     "eslint-plugin-flowtype": "^2.39.1",
     "eslint-plugin-import": "^2.2.0",
     "eslint-plugin-jsx-a11y": "^5.1.1",
+    "eslint-plugin-prettier": "^2.3.1",
     "eslint-plugin-react": "^7.4.0",
     "gitbook-cli": "^2.3.2",
     "glob": "^7.1.1",
     "jest": "^21.2.1",
+    "prettier": "^1.7.4",
     "rimraf": "^2.6.2",
     "rollup": "^0.50.0",
     "rollup-plugin-babel": "^3.0.2",

package.json

@@ -17,7 +17,7 @@ import compose from './compose'
  * @returns {Function} A store enhancer applying the middleware.
  */
 export default function applyMiddleware(...middlewares) {
-  return (createStore) => (...args) => {
+  return createStore => (...args) => {
     const store = createStore(...args)
     let dispatch = store.dispatch
     let chain = []

src/applyMiddleware.js

@@ -1,5 +1,7 @@
 function bindActionCreator(actionCreator, dispatch) {
-  return function() { return dispatch(actionCreator.apply(this, arguments)) }
+  return function() {
+    return dispatch(actionCreator.apply(this, arguments))
+  }
 }
 
 /**
@@ -30,8 +32,11 @@ export default function bindActionCreators(actionCreators, dispatch) {
 
   if (typeof actionCreators !== 'object' || actionCreators === null) {
     throw new Error(
-      `bindActionCreators expected an object or a function, instead received ${actionCreators === null ? 'null' : typeof actionCreators}. ` +
-      `Did you write "import ActionCreators from" instead of "import * as ActionCreators from"?`
+      `bindActionCreators expected an object or a function, instead received ${actionCreators ===
+      null
+        ? 'null'
+        : typeof actionCreators}. ` +
+        `Did you write "import ActionCreators from" instead of "import * as ActionCreators from"?`
     )
   }
 

src/bindActionCreators.js

@@ -4,7 +4,8 @@ import warning from './utils/warning'
 
 function getUndefinedStateErrorMessage(key, action) {
   const actionType = action && action.type
-  const actionDescription = (actionType && `action "${String(actionType)}"`) || 'an action'
+  const actionDescription =
+    (actionType && `action "${String(actionType)}"`) || 'an action'
 
   return (
     `Given ${actionDescription}, reducer "${key}" returned undefined. ` +
@@ -13,11 +14,17 @@ function getUndefinedStateErrorMessage(key, action) {
   )
 }
 
-function getUnexpectedStateShapeWarningMessage(inputState, reducers, action, unexpectedKeyCache) {
+function getUnexpectedStateShapeWarningMessage(
+  inputState,
+  reducers,
+  action,
+  unexpectedKeyCache
+) {
   const reducerKeys = Object.keys(reducers)
-  const argumentName = action && action.type === ActionTypes.INIT ?
-    'preloadedState argument passed to createStore' :
-    'previous state received by the reducer'
+  const argumentName =
+    action && action.type === ActionTypes.INIT
+      ? 'preloadedState argument passed to createStore'
+      : 'previous state received by the reducer'
 
   if (reducerKeys.length === 0) {
     return (
@@ -29,15 +36,14 @@ function getUnexpectedStateShapeWarningMessage(inputState, reducers, action, une
   if (!isPlainObject(inputState)) {
     return (
       `The ${argumentName} has unexpected type of "` +
-      ({}).toString.call(inputState).match(/\s([a-z|A-Z]+)/)[1] +
+      {}.toString.call(inputState).match(/\s([a-z|A-Z]+)/)[1] +
       `". Expected argument to be an object with the following ` +
       `keys: "${reducerKeys.join('", "')}"`
     )
   }
 
-  const unexpectedKeys = Object.keys(inputState).filter(key =>
-    !reducers.hasOwnProperty(key) &&
-    !unexpectedKeyCache[key]
+  const unexpectedKeys = Object.keys(inputState).filter(
+    key => !reducers.hasOwnProperty(key) && !unexpectedKeyCache[key]
   )
 
   unexpectedKeys.forEach(key => {
@@ -62,22 +68,28 @@ function assertReducerShape(reducers) {
     if (typeof initialState === 'undefined') {
       throw new Error(
         `Reducer "${key}" returned undefined during initialization. ` +
-        `If the state passed to the reducer is undefined, you must ` +
-        `explicitly return the initial state. The initial state may ` +
-        `not be undefined. If you don't want to set a value for this reducer, ` +
-        `you can use null instead of undefined.`
+          `If the state passed to the reducer is undefined, you must ` +
+          `explicitly return the initial state. The initial state may ` +
+          `not be undefined. If you don't want to set a value for this reducer, ` +
+          `you can use null instead of undefined.`
       )
     }
 
-    const type = '@@redux/PROBE_UNKNOWN_ACTION_' + Math.random().toString(36).substring(7).split('').join('.')
+    const type =
+      '@@redux/PROBE_UNKNOWN_ACTION_' +
+      Math.random()
+        .toString(36)
+        .substring(7)
+        .split('')
+        .join('.')
     if (typeof reducer(undefined, { type }) === 'undefined') {
       throw new Error(
         `Reducer "${key}" returned undefined when probed with a random type. ` +
-        `Don't try to handle ${ActionTypes.INIT} or other actions in "redux/*" ` +
-        `namespace. They are considered private. Instead, you must return the ` +
-        `current state for any unknown actions, unless it is undefined, ` +
-        `in which case you must return the initial state, regardless of the ` +
-        `action type. The initial state may not be undefined, but can be null.`
+          `Don't try to handle ${ActionTypes.INIT} or other actions in "redux/*" ` +
+          `namespace. They are considered private. Instead, you must return the ` +
+          `current state for any unknown actions, unless it is undefined, ` +
+          `in which case you must return the initial state, regardless of the ` +
+          `action type. The initial state may not be undefined, but can be null.`
       )
     }
   })
@@ -135,7 +147,12 @@ export default function combineReducers(reducers) {
     }
 
     if (process.env.NODE_ENV !== 'production') {
-      const warningMessage = getUnexpectedStateShapeWarningMessage(state, finalReducers, action, unexpectedKeyCache)
+      const warningMessage = getUnexpectedStateShapeWarningMessage(
+        state,
+        finalReducers,
+        action,
+        unexpectedKeyCache
+      )
       if (warningMessage) {
         warning(warningMessage)
       }

src/combineReducers.js

@@ -150,14 +150,14 @@ export default function createStore(reducer, preloadedState, enhancer) {
     if (!isPlainObject(action)) {
       throw new Error(
         'Actions must be plain objects. ' +
-        'Use custom middleware for async actions.'
+          'Use custom middleware for async actions.'
       )
     }
 
     if (typeof action.type === 'undefined') {
       throw new Error(
         'Actions may not have an undefined "type" property. ' +
-        'Have you misspelled a constant?'
+          'Have you misspelled a constant?'
       )
     }
 
@@ -172,7 +172,7 @@ export default function createStore(reducer, preloadedState, enhancer) {
       isDispatching = false
     }
 
-    const listeners = currentListeners = nextListeners
+    const listeners = (currentListeners = nextListeners)
     for (let i = 0; i < listeners.length; i++) {
       const listener = listeners[i]
       listener()

src/createStore.js

@@ -17,11 +17,11 @@ if (
   isCrushed.name !== 'isCrushed'
 ) {
   warning(
-    'You are currently using minified code outside of NODE_ENV === \'production\'. ' +
-    'This means that you are running a slower development build of Redux. ' +
-    'You can use loose-envify (https://github.com/zertosh/loose-envify) for browserify ' +
-    'or DefinePlugin for webpack (http://stackoverflow.com/questions/30030031) ' +
-    'to ensure you have the correct code for your production build.'
+    "You are currently using minified code outside of NODE_ENV === 'production'. " +
+      'This means that you are running a slower development build of Redux. ' +
+      'You can use loose-envify (https://github.com/zertosh/loose-envify) for browserify ' +
+      'or DefinePlugin for webpack (http://stackoverflow.com/questions/30030031) ' +
+      'to ensure you have the correct code for your production build.'
   )
 }
 

src/index.js

@@ -15,7 +15,5 @@ export default function warning(message) {
     // "break on all exceptions" in your console,
     // it would pause the execution at this line.
     throw new Error(message)
-  /* eslint-disable no-empty */
-  } catch (e) { }
-  /* eslint-enable no-empty */
+  } catch (e) {} // eslint-disable-line no-empty
 }

src/utils/warning.js

@@ -23,7 +23,10 @@ describe('applyMiddleware', () => {
     expect(spy.mock.calls[0][0]).toHaveProperty('getState')
     expect(spy.mock.calls[0][0]).toHaveProperty('dispatch')
 
-    expect(store.getState()).toEqual([ { id: 1, text: 'Use Redux' }, { id: 2, text: 'Flux FTW!' } ])
+    expect(store.getState()).toEqual([
+      { id: 1, text: 'Use Redux' },
+      { id: 2, text: 'Flux FTW!' }
+    ])
   })
 
   it('passes recursive dispatches through the middleware chain', () => {
@@ -92,25 +95,28 @@ describe('applyMiddleware', () => {
     })
   })
 
-    it('passes through all arguments of dispatch calls from within middleware', () => {
-        const spy = jest.fn()
-        const testCallArgs = ['test']
-        function multiArgMiddleware() {
-            return next => (action, callArgs) => {
-                if (Array.isArray(callArgs)) {
-                    return action(...callArgs)
-                }
-                return next(action)
-            }
-        }
-        function dummyMiddleware({ dispatch }) {
-            return next => action => dispatch(action, testCallArgs)
+  it('passes through all arguments of dispatch calls from within middleware', () => {
+    const spy = jest.fn()
+    const testCallArgs = ['test']
+    function multiArgMiddleware() {
+      return next => (action, callArgs) => {
+        if (Array.isArray(callArgs)) {
+          return action(...callArgs)
         }
+        return next(action)
+      }
+    }
+    function dummyMiddleware({ dispatch }) {
+      return next => action => dispatch(action, testCallArgs)
+    }
 
-        const store = createStore(reducers.todos, applyMiddleware(multiArgMiddleware, dummyMiddleware))
-        store.dispatch(spy)
-        expect(spy.mock.calls[0]).toEqual(testCallArgs)
-    })
+    const store = createStore(
+      reducers.todos,
+      applyMiddleware(multiArgMiddleware, dummyMiddleware)
+    )
+    store.dispatch(spy)
+    expect(spy.mock.calls[0]).toEqual(testCallArgs)
+  })
 
   it('keeps unwrapped dispatch available while middleware is initializing', () => {
     // This is documenting the existing behavior in Redux 3.x.