From 3690f04e4a84fa43c43b39e5e3884a20c9e0efe3 Mon Sep 17 00:00:00 2001
From: Yamagishi Kazutoshi <ykzts@desire.sh>
Date: Sun, 11 Jun 2017 17:42:42 +0900
Subject: [PATCH] Remove comments for eslint-disable (#3691)

---
 .eslintrc.yml                                               | 5 ++++-
 .../features/notifications/components/notification.js       | 6 ++++--
 app/javascript/mastodon/middleware/errors.js                | 2 +-
 app/javascript/packs/application.js                         | 2 +-
 app/javascript/packs/public.js                              | 2 +-
 config/webpack/production.js                                | 2 --
 config/webpack/shared.js                                    | 3 ---
 config/webpack/translationRunner.js                         | 2 --
 8 files changed, 11 insertions(+), 13 deletions(-)

diff --git a/.eslintrc.yml b/.eslintrc.yml
index 049d4ecf34..2fb54ae669 100644
--- a/.eslintrc.yml
+++ b/.eslintrc.yml
@@ -43,7 +43,10 @@ rules:
   - prefer-single
   no-catch-shadow: error
   no-cond-assign: error
-  no-console: warn
+  no-console:
+  - warn
+  - allow:
+    - error
   no-fallthrough: error
   no-irregular-whitespace: error
   no-mixed-spaces-and-tabs: warn
diff --git a/app/javascript/mastodon/features/notifications/components/notification.js b/app/javascript/mastodon/features/notifications/components/notification.js
index 9afdf6f4c3..6ec4d5dc67 100644
--- a/app/javascript/mastodon/features/notifications/components/notification.js
+++ b/app/javascript/mastodon/features/notifications/components/notification.js
@@ -65,11 +65,11 @@ class Notification extends ImmutablePureComponent {
     );
   }
 
-  render () { // eslint-disable-line consistent-return
+  render () {
     const { notification } = this.props;
     const account          = notification.get('account');
     const displayName      = account.get('display_name').length > 0 ? account.get('display_name') : account.get('username');
-    const displayNameHTML = { __html: emojify(escapeTextContentForBrowser(displayName)) };
+    const displayNameHTML  = { __html: emojify(escapeTextContentForBrowser(displayName)) };
     const link             = <Permalink className='notification__display-name' href={account.get('url')} title={account.get('acct')} to={`/accounts/${account.get('id')}`} dangerouslySetInnerHTML={displayNameHTML} />;
 
     switch(notification.get('type')) {
@@ -82,6 +82,8 @@ class Notification extends ImmutablePureComponent {
     case 'reblog':
       return this.renderReblog(notification, link);
     }
+
+    return null;
   }
 
 }
diff --git a/app/javascript/mastodon/middleware/errors.js b/app/javascript/mastodon/middleware/errors.js
index 9a51257cb8..4aca75f1e3 100644
--- a/app/javascript/mastodon/middleware/errors.js
+++ b/app/javascript/mastodon/middleware/errors.js
@@ -22,7 +22,7 @@ export default function errorsMiddleware() {
 
           dispatch(showAlert(title, message));
         } else {
-          console.error(action.error); // eslint-disable-line no-console
+          console.error(action.error);
           dispatch(showAlert('Oops!', 'An unexpected error occurred.'));
         }
       }
diff --git a/app/javascript/packs/application.js b/app/javascript/packs/application.js
index 8932dc5138..63c5d62720 100644
--- a/app/javascript/packs/application.js
+++ b/app/javascript/packs/application.js
@@ -2,5 +2,5 @@ import main from '../mastodon/main';
 import loadPolyfills from '../mastodon/load_polyfills';
 
 loadPolyfills().then(main).catch(e => {
-  console.error(e); // eslint-disable-line no-console
+  console.error(e);
 });
diff --git a/app/javascript/packs/public.js b/app/javascript/packs/public.js
index 4c0a3d6939..a0e511b0a3 100644
--- a/app/javascript/packs/public.js
+++ b/app/javascript/packs/public.js
@@ -93,5 +93,5 @@ function main() {
 }
 
 loadPolyfills().then(main).catch(error => {
-  console.log(error); // eslint-disable-line no-console
+  console.error(error);
 });
diff --git a/config/webpack/production.js b/config/webpack/production.js
index c93dd55e30..7554640614 100644
--- a/config/webpack/production.js
+++ b/config/webpack/production.js
@@ -1,7 +1,5 @@
 // Note: You must restart bin/webpack-dev-server for changes to take effect
 
-/* eslint global-require: 0 */
-
 const webpack = require('webpack');
 const merge = require('webpack-merge');
 const CompressionPlugin = require('compression-webpack-plugin');
diff --git a/config/webpack/shared.js b/config/webpack/shared.js
index 0f767fb473..8fd80a00d5 100644
--- a/config/webpack/shared.js
+++ b/config/webpack/shared.js
@@ -1,8 +1,5 @@
 // Note: You must restart bin/webpack-dev-server for changes to take effect
 
-/* eslint global-require: 0 */
-/* eslint import/no-dynamic-require: 0 */
-
 const { existsSync } = require('fs');
 const webpack = require('webpack');
 const { basename, dirname, join, relative, resolve, sep } = require('path');
diff --git a/config/webpack/translationRunner.js b/config/webpack/translationRunner.js
index 79bb6980f5..097099b48c 100644
--- a/config/webpack/translationRunner.js
+++ b/config/webpack/translationRunner.js
@@ -38,7 +38,6 @@ const validateLanguages = (languages, validators) => {
   }, []);
 
   if (invalidLanguages.length > 0) {
-    // eslint-disable-next-line no-console
     console.error(`
 Error: Specified invalid LANGUAGES:
 ${invalidLanguages.map(({ language, error }) => `* ${language}: ${error.message}`).join('\n')}
@@ -73,7 +72,6 @@ ${availableLanguages.join(', ')}
 
 // check if message directory exists
 if (!fs.existsSync(messagesDirectory)) {
-  // eslint-disable-next-line no-console
   console.error(`
 Error: messagesDirectory not exists
 (${messagesDirectory})