linting

Author: Judahmeek

docs/guides/rails-webpacker-react-integration-options.md

@@ -132,10 +132,7 @@ You can see an example commit of adding this [here](https://github.com/shakacode
 
    //plugins
    if (isWebpackDevServer) {
-     environment.plugins.append(
-       'ReactRefreshWebpackPlugin',
-       new ReactRefreshWebpackPlugin({}),
-     );
+     environment.plugins.append('ReactRefreshWebpackPlugin', new ReactRefreshWebpackPlugin({}));
    }
    ```
 

spec/dummy/config/webpack/development.js

@@ -1,6 +1,6 @@
 process.env.NODE_ENV = process.env.NODE_ENV || 'development';
 
-const { devServer, inliningCss } = require('shakapacker');
+const { inliningCss } = require('shakapacker');
 
 const webpackConfig = require('./webpackConfig');
 
@@ -12,9 +12,7 @@ const developmentEnvOnly = (clientWebpackConfig, _serverWebpackConfig) => {
 
     // eslint-disable-next-line global-require
     const ReactRefreshWebpackPlugin = require('@pmmmwh/react-refresh-webpack-plugin');
-    clientWebpackConfig.plugins.push(
-      new ReactRefreshWebpackPlugin({}),
-    );
+    clientWebpackConfig.plugins.push(new ReactRefreshWebpackPlugin({}));
   }
 };
 

spec/dummy/tests/react-on-rails.import.test.js

@@ -1,5 +1,7 @@
 import ReactOnRails from 'react-on-rails';
 
 test('ReactOnRails', () => {
-  ReactOnRails.register({});
+  expect(() => {
+    ReactOnRails.register({});
+  }).not.toThrow();
 });

spec/dummy/tests/react-on-rails.require.test.js

@@ -1,5 +1,7 @@
 const ReactOnRails = require('react-on-rails').default;
 
 test('ReactOnRails', () => {
-  ReactOnRails.register({});
+  expect(() => {
+    ReactOnRails.register({});
+  }).not.toThrow();
 });