Prettier 2.0

Author: timdorr

Diff for: .prettierrc.json

@@ -1,5 +1,7 @@
 {
   "semi": false,
   "singleQuote": true,
-  "tabWidth": 2
+  "tabWidth": 2,
+  "trailingComma": "none",
+  "arrowParens": "avoid"
 }

Diff for: docs/advanced/AsyncActions.md

@@ -357,7 +357,7 @@ export function fetchPosts(subreddit) {
   // It passes the dispatch method as an argument to the function,
   // thus making it able to dispatch actions itself.
 
-  return function(dispatch) {
+  return function (dispatch) {
     // First dispatch: the app state is updated to inform
     // that the API call is starting.
 

Diff for: docs/api/applyMiddleware.md

@@ -109,7 +109,7 @@ function makeASandwichWithSecretSauce(forPerson) {
   // Invert control!
   // Return a function that accepts `dispatch` so we can dispatch later.
   // Thunk middleware knows how to turn thunk async actions into actions.
-  return function(dispatch) {
+  return function (dispatch) {
     return fetchSecretSauce().then(
       sauce => dispatch(makeASandwich(forPerson, sauce)),
       error => dispatch(apologize('The Sandwich Shop', forPerson, error))
@@ -131,7 +131,7 @@ store.dispatch(makeASandwichWithSecretSauce('My wife')).then(() => {
 // actions and async actions from other action creators,
 // and I can build my control flow with Promises.
 function makeSandwichesForEverybody() {
-  return function(dispatch, getState) {
+  return function (dispatch, getState) {
     if (!getState().sandwiches.isShopOpen) {
       // You don't have to return Promises, but it's a handy convention
       // so the caller can always call .then() on async dispatch result.

Diff for: docs/introduction/Ecosystem.md

@@ -365,10 +365,7 @@ const newData = update(myData, {
 Simpler alternative to immutability-helpers and Immutable.js
 
 ```js
-const newObj = immutable(obj)
-  .set('a.b', 'f')
-  .del(['a', 'c', 0])
-  .value()
+const newObj = immutable(obj).set('a.b', 'f').del(['a', 'c', 0]).value()
 ```
 
 **[debitoor/dot-prop-immutable](https://github.com/debitoor/dot-prop-immutable)** <br />

Diff for: docs/recipes/ImplementingUndoHistory.md

@@ -267,7 +267,7 @@ A reducer enhancer that doesn't do anything looks like this:
 
 ```js
 function doNothingWith(reducer) {
-  return function(state, action) {
+  return function (state, action) {
     // Just call the passed reducer
     return reducer(state, action)
   }
@@ -278,7 +278,7 @@ A reducer enhancer that combines other reducers might look like this:
 
 ```js
 function combineReducers(reducers) {
-  return function(state = {}, action) {
+  return function (state = {}, action) {
     return Object.keys(reducers).reduce((nextState, key) => {
       // Call every reducer with the part of the state it manages
       nextState[key] = reducers[key](state[key], action)
@@ -302,7 +302,7 @@ function undoable(reducer) {
   }
 
   // Return a reducer that handles undo and redo
-  return function(state = initialState, action) {
+  return function (state = initialState, action) {
     const { past, present, future } = state
 
     switch (action.type) {

Diff for: docs/recipes/ReducingBoilerplate.md

@@ -92,7 +92,7 @@ function addTodoWithoutCheck(text) {
 export function addTodo(text) {
   // This form is allowed by Redux Thunk middleware
   // described below in “Async Action Creators” section.
-  return function(dispatch, getState) {
+  return function (dispatch, getState) {
     if (getState().todos.length === 3) {
       // Exit early
       return
@@ -138,7 +138,7 @@ You can always write a function that generates an action creator:
 
 ```js
 function makeActionCreator(type, ...argNames) {
-  return function(...args) {
+  return function (...args) {
     const action = { type }
     argNames.forEach((arg, index) => {
       action[argNames[index]] = args[index]
@@ -269,7 +269,7 @@ Consider the code above rewritten with [redux-thunk](https://github.com/gaearon/
 ```js
 export function loadPosts(userId) {
   // Interpreted by the thunk middleware:
-  return function(dispatch, getState) {
+  return function (dispatch, getState) {
     const { posts } = getState()
     if (posts[userId]) {
       // There is cached data! Don't do anything.
@@ -473,7 +473,7 @@ const TodoStore = Object.assign({}, EventEmitter.prototype, {
   }
 })
 
-AppDispatcher.register(function(action) {
+AppDispatcher.register(function (action) {
   switch (action.type) {
     case ActionTypes.ADD_TODO:
       const text = action.text.trim()

Diff for: src/bindActionCreators.ts

@@ -9,7 +9,7 @@ function bindActionCreator<A extends AnyAction = AnyAction>(
   actionCreator: ActionCreator<A>,
   dispatch: Dispatch
 ) {
-  return function(this: any, ...args: any[]) {
+  return function (this: any, ...args: any[]) {
     return dispatch(actionCreator.apply(this, args))
   }
 }

Diff for: src/utils/actionTypes.ts

@@ -6,11 +6,7 @@
  */
 
 const randomString = () =>
-  Math.random()
-    .toString(36)
-    .substring(7)
-    .split('')
-    .join('.')
+  Math.random().toString(36).substring(7).split('').join('.')
 
 const ActionTypes = {
   INIT: `@@redux/INIT${/* #__PURE__ */ randomString()}`,

Diff for: test/combineReducers.spec.ts

@@ -322,12 +322,12 @@ describe('Utils', () => {
       console.error = preSpy
     })
 
-    describe('With Replace Reducers', function() {
+    describe('With Replace Reducers', function () {
       const foo = (state = {}) => state
       const bar = (state = {}) => state
       const ACTION = { type: 'ACTION' }
 
-      it('should return an updated state when additional reducers are passed to combineReducers', function() {
+      it('should return an updated state when additional reducers are passed to combineReducers', function () {
         const originalCompositeReducer = combineReducers({ foo })
         const store = createStore(originalCompositeReducer)
 
@@ -342,7 +342,7 @@ describe('Utils', () => {
         expect(nextState).not.toBe(initialState)
       })
 
-      it('should return an updated state when reducers passed to combineReducers are changed', function() {
+      it('should return an updated state when reducers passed to combineReducers are changed', function () {
         const baz = (state = {}) => state
 
         const originalCompositeReducer = combineReducers({ foo, bar })
@@ -359,7 +359,7 @@ describe('Utils', () => {
         expect(nextState).not.toBe(initialState)
       })
 
-      it('should return the same state when reducers passed to combineReducers not changed', function() {
+      it('should return the same state when reducers passed to combineReducers not changed', function () {
         const originalCompositeReducer = combineReducers({ foo, bar })
         const store = createStore(originalCompositeReducer)
 
@@ -374,7 +374,7 @@ describe('Utils', () => {
         expect(nextState).toBe(initialState)
       })
 
-      it('should return an updated state when one of more reducers passed to the combineReducers are removed', function() {
+      it('should return an updated state when one of more reducers passed to the combineReducers are removed', function () {
         const originalCompositeReducer = combineReducers({ foo, bar })
         const store = createStore(originalCompositeReducer)
 

Diff for: test/createStore.spec.ts

@@ -422,7 +422,7 @@ describe('createStore', () => {
 
   it('does not leak private listeners array', done => {
     const store = createStore(reducers.todos)
-    store.subscribe(function() {
+    store.subscribe(function () {
       expect(this).toBe(undefined)
       done()
     })
@@ -664,19 +664,19 @@ describe('createStore', () => {
         const store = createStore(() => {})
         const obs = store[$$observable]()
 
-        expect(function() {
+        expect(function () {
           obs.subscribe()
         }).toThrowError(new TypeError('Expected the observer to be an object.'))
 
-        expect(function() {
+        expect(function () {
           obs.subscribe(null)
         }).toThrowError(new TypeError('Expected the observer to be an object.'))
 
-        expect(function() {
+        expect(function () {
           obs.subscribe(() => {})
         }).toThrowError(new TypeError('Expected the observer to be an object.'))
 
-        expect(function() {
+        expect(function () {
           obs.subscribe({})
         }).not.toThrow()
       })