Updated eslint-config-shakacode to 13.2.0

Big change is that trailing commas are NO LONGER the default.

Autofixer makes this easy!

Be sure to run

npm run lint -- --fix

and you will be updated!

Author: justin808

client/app/bundles/comments/components/CommentBox/CommentList/Comment/Comment.spec.jsx

@@ -11,7 +11,7 @@ const {
 describe('Comment', () => {
   it('renders an author and comment with proper css classes', () => {
     const component = renderIntoDocument(
-      <Comment author="Frank" text="Hi!" />
+      <Comment author="Frank" text="Hi!" />,
     );
 
     // TODO: Setup testing of CSS Modules classNames
@@ -25,7 +25,7 @@ describe('Comment', () => {
 
   it('shows the author', () => {
     const component = renderIntoDocument(
-      <Comment author="Frank" text="Hi!" />
+      <Comment author="Frank" text="Hi!" />,
     );
 
     const author = findRenderedDOMComponentWithClass(component, 'js-comment-author');
@@ -34,7 +34,7 @@ describe('Comment', () => {
 
   it('shows the comment text in markdown', () => {
     const component = renderIntoDocument(
-      <Comment author="Frank" text="Hi!" />
+      <Comment author="Frank" text="Hi!" />,
     );
 
     const comment = findRenderedDOMComponentWithClass(component, 'js-comment-text');

client/app/bundles/comments/components/CommentBox/CommentList/CommentList.jsx

@@ -44,7 +44,7 @@ export default class CommentList extends BaseComponent {
         key={$$comment.get('id') || index}
         author={$$comment.get('author')}
         text={$$comment.get('text')}
-      />
+      />,
     );
 
     return (

client/app/bundles/comments/components/CommentBox/CommentList/CommentList.spec.jsx

@@ -36,7 +36,7 @@ describe('CommentList', () => {
       <CommentList
         $$comments={comments}
         cssTransitionGroupClassNames={cssTransitionGroupClassNames}
-      />
+      />,
     );
     const list = scryRenderedComponentsWithType(component, Comment);
     expect(list.length).to.equal(2);
@@ -49,7 +49,7 @@ describe('CommentList', () => {
       <CommentList
         $$comments={comments} error="zomg"
         cssTransitionGroupClassNames={cssTransitionGroupClassNames}
-      />
+      />,
     );
 
     const alert = findRenderedDOMComponentWithTag(component, 'strong');

client/app/bundles/comments/reducers/commentsReducer.js

@@ -37,7 +37,7 @@ export default function commentsReducer($$state = $$initialState, action = null)
         state
           .updateIn(
             ['$$comments'],
-            $$comments => $$comments.unshift(Immutable.fromJS(comment))
+            $$comments => $$comments.unshift(Immutable.fromJS(comment)),
           )
           .merge({
             submitCommentError: null,

client/app/bundles/comments/startup/ClientRouterApp.jsx

@@ -13,7 +13,7 @@ export default (_props, _railsContext) => {
   // Create an enhanced history that syncs navigation events with the store
   const history = syncHistoryWithStore(
     browserHistory,
-    store
+    store,
   );
 
   return (

client/app/bundles/comments/startup/ClientRouterAppExpress.jsx

@@ -18,7 +18,7 @@ const RouterAppExpress = (_props, _railsContext) => {
   // Create an enhanced history that syncs navigation events with the store
   const history = syncHistoryWithStore(
     browserHistory,
-    store
+    store,
   );
 
   return (

client/app/bundles/comments/startup/serverRegistration.jsx

@@ -14,7 +14,7 @@ ReactOnRails.register(
     RouterApp,
     NavigationBarApp,
     SimpleCommentScreen,
-  }
+  },
 );
 
 ReactOnRails.registerStore({

client/app/bundles/comments/store/commentsStore.js

@@ -15,7 +15,7 @@ export default (props, railsContext) => {
 
   const reducer = combineReducers(reducers);
   const composedStore = compose(
-    applyMiddleware(thunkMiddleware, loggerMiddleware)
+    applyMiddleware(thunkMiddleware, loggerMiddleware),
   );
 
   return composedStore(createStore)(reducer, initialState);

client/app/bundles/comments/store/routerCommentsStore.js

@@ -24,7 +24,7 @@ export default (props, railsContext) => {
 
   // Sync dispatched route actions to the history
   const finalCreateStore = compose(
-    applyMiddleware(thunkMiddleware, loggerMiddleware)
+    applyMiddleware(thunkMiddleware, loggerMiddleware),
   )(createStore);
 
   return finalCreateStore(reducer, initialState);

client/app/libs/middlewares/loggerMiddleware.js

@@ -15,7 +15,6 @@ export default function logger({ getState }) {
       readableState[storeItem] = (
         state[storeItem].toJS ? state[storeItem].toJS() : state[storeItem]
       );
-      return;
     });
 
     console.log('state after dispatch', readableState);

client/package.json

@@ -94,7 +94,7 @@
     "chai": "^3.5.0",
     "chai-immutable": "^1.6.0",
     "eslint": "^3.10.2",
-    "eslint-config-shakacode": "^13.1.0",
+    "eslint-config-shakacode": "^13.2.0-beta.1",
     "eslint-import-resolver-webpack": "^0.7.0",
     "eslint-plugin-import": "^2.2.0",
     "eslint-plugin-jsx-a11y": "^2.2.3",

client/server-rails-hot.js

@@ -30,6 +30,6 @@ const devServer = new WebpackDevServer(compiler, {
 devServer.listen(hotRailsPort, 'localhost', (err) => {
   if (err) console.error(err);
   console.log(
-    `=> 🔥  Webpack development server is running on port ${hotRailsPort}`
+    `=> 🔥  Webpack development server is running on port ${hotRailsPort}`,
   );
 });

client/webpack.client.express.config.js

@@ -16,7 +16,7 @@ config.entry.app.push(
 
   // Webpack dev server
   `webpack-dev-server/client?http://localhost:${hotPort}`,
-  'webpack/hot/dev-server'
+  'webpack/hot/dev-server',
 );
 
 config.output = {
@@ -27,7 +27,7 @@ config.output = {
 };
 config.plugins.unshift(
   new webpack.HotModuleReplacementPlugin(),
-  new webpack.NoErrorsPlugin()
+  new webpack.NoErrorsPlugin(),
 );
 config.devtool = 'eval-source-map';
 
@@ -71,7 +71,7 @@ config.module.loaders.push(
       'sass',
       'sass-resources',
     ],
-  }
+  },
 );
 
 module.exports = config;

client/webpack.client.rails.build.config.js

@@ -21,7 +21,7 @@ config.entry.vendor.unshift(
   'jquery-ujs',
 
   // Configures extractStyles to be true if NODE_ENV is production
-  'bootstrap-loader/extractStyles'
+  'bootstrap-loader/extractStyles',
 );
 
 // See webpack.common.config for adding modules common to both the webpack dev server and rails
@@ -37,7 +37,7 @@ config.module.loaders.push(
     loader: ExtractTextPlugin.extract(
       'style',
       'css?minimize&modules&importLoaders=1&localIdentName=[name]__[local]__[hash:base64:5]' +
-      '!postcss'
+      '!postcss',
     ),
   },
   {
@@ -47,7 +47,7 @@ config.module.loaders.push(
       'css?minimize&modules&importLoaders=3&localIdentName=[name]__[local]__[hash:base64:5]' +
       '!postcss' +
       '!sass' +
-      '!sass-resources'
+      '!sass-resources',
     ),
   },
   {
@@ -57,12 +57,12 @@ config.module.loaders.push(
   {
     test: require.resolve('jquery-ujs'),
     loader: 'imports?jQuery=jquery',
-  }
+  },
 );
 
 config.plugins.push(
   new ExtractTextPlugin('[name]-bundle.css', { allChunks: true }),
-  new webpack.optimize.DedupePlugin()
+  new webpack.optimize.DedupePlugin(),
 );
 
 if (devBuild) {

client/webpack.client.rails.hot.config.js

@@ -12,13 +12,13 @@ const hotRailsPort = process.env.HOT_RAILS_PORT || 3500;
 
 config.entry.app.push(
   `webpack-dev-server/client?http://localhost:${hotRailsPort}`,
-  'webpack/hot/only-dev-server'
+  'webpack/hot/only-dev-server',
 );
 
 // These are Rails specific
 config.entry.vendor.push(
   'jquery-ujs',
-  'bootstrap-loader'
+  'bootstrap-loader',
 );
 
 config.output = {
@@ -71,12 +71,12 @@ config.module.loaders.push(
   {
     test: require.resolve('jquery-ujs'),
     loader: 'imports?jQuery=jquery',
-  }
+  },
 );
 
 config.plugins.push(
   new webpack.HotModuleReplacementPlugin(),
-  new webpack.NoErrorsPlugin()
+  new webpack.NoErrorsPlugin(),
 );
 
 config.devtool = 'eval-source-map';