From 64fb7a2d0c50b98ec123a4d8fda4a7620bfdeefd Mon Sep 17 00:00:00 2001
From: Endi <endiliey@gmail.com>
Date: Thu, 9 May 2019 20:01:48 +0700
Subject: [PATCH] chore(v2): tweak eslint config (#1445)

---
 .eslintrc.js                                           | 10 +++++++---
 packages/docusaurus/src/client/App.js                  |  8 ++++----
 packages/docusaurus/src/client/clientEntry.js          |  2 +-
 packages/docusaurus/src/client/docusaurus.js           |  2 +-
 packages/docusaurus/src/client/serverEntry.js          |  2 +-
 .../src/client/theme-fallback/Layout/index.js          |  4 ++--
 website/pages/index.js                                 |  5 +++++
 7 files changed, 21 insertions(+), 12 deletions(-)

diff --git a/.eslintrc.js b/.eslintrc.js
index 9b5f970108..251595e0b7 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -25,7 +25,11 @@ module.exports = {
   rules: {
     'class-methods-use-this': OFF, // It's a way of allowing private variables.
     'func-names': OFF,
-    'import/no-unresolved': WARNING, // Because it couldn't resolve webpack alias.
+    // Ignore certain webpack alias because it can't be resolved
+    'import/no-unresolved': [
+      ERROR,
+      {ignore: ['^@theme', '^@docusaurus', '^@generated']},
+    ],
     'header/header': [
       ERROR,
       'block',
@@ -42,8 +46,8 @@ module.exports = {
         ' ',
       ],
     ],
-    'jsx-a11y/click-events-have-key-events': OFF, // Revisit in future™
-    'jsx-a11y/no-noninteractive-element-interactions': OFF, // Revisit in future™
+    'jsx-a11y/click-events-have-key-events': WARNING,
+    'jsx-a11y/no-noninteractive-element-interactions': WARNING,
     'no-console': OFF,
     'no-underscore-dangle': OFF,
     'react/jsx-closing-bracket-location': OFF, // Conflicts with Prettier.
diff --git a/packages/docusaurus/src/client/App.js b/packages/docusaurus/src/client/App.js
index 7d41a2c338..f16ccb474b 100644
--- a/packages/docusaurus/src/client/App.js
+++ b/packages/docusaurus/src/client/App.js
@@ -8,10 +8,10 @@
 import React from 'react';
 import {renderRoutes} from 'react-router-config';
 
-import Head from '@docusaurus/Head'; // eslint-disable-line
-import routes from '@generated/routes'; // eslint-disable-line
-import siteConfig from '@generated/docusaurus.config'; //eslint-disable-line
-import DocusaurusContext from '@docusaurus/context'; // eslint-disable-line
+import Head from '@docusaurus/Head';
+import routes from '@generated/routes';
+import siteConfig from '@generated/docusaurus.config';
+import DocusaurusContext from '@docusaurus/context';
 import PendingNavigation from './PendingNavigation';
 
 function App() {
diff --git a/packages/docusaurus/src/client/clientEntry.js b/packages/docusaurus/src/client/clientEntry.js
index 967aec9e65..28fc5408c6 100644
--- a/packages/docusaurus/src/client/clientEntry.js
+++ b/packages/docusaurus/src/client/clientEntry.js
@@ -9,10 +9,10 @@ import React from 'react';
 import {hydrate, render} from 'react-dom';
 import {BrowserRouter} from 'react-router-dom';
 
+import routes from '@generated/routes';
 import App from './App';
 import preload from './preload';
 import docusaurus from './docusaurus';
-import routes from '@generated/routes'; // eslint-disable-line
 
 // Client-side render (e.g: running in browser) to become single-page application (SPA).
 if (typeof window !== 'undefined' && typeof document !== 'undefined') {
diff --git a/packages/docusaurus/src/client/docusaurus.js b/packages/docusaurus/src/client/docusaurus.js
index cc4a917128..5497872d63 100644
--- a/packages/docusaurus/src/client/docusaurus.js
+++ b/packages/docusaurus/src/client/docusaurus.js
@@ -6,7 +6,7 @@
  */
 import {matchRoutes} from 'react-router-config';
 import routesChunkNames from '@generated/routesChunkNames';
-import routes from '@generated/routes'; // eslint-disable-line
+import routes from '@generated/routes';
 import prefetchHelper from './prefetch';
 import preloadHelper from './preload';
 import flat from './flat';
diff --git a/packages/docusaurus/src/client/serverEntry.js b/packages/docusaurus/src/client/serverEntry.js
index a60c6cd38f..ae5304a7ca 100644
--- a/packages/docusaurus/src/client/serverEntry.js
+++ b/packages/docusaurus/src/client/serverEntry.js
@@ -15,7 +15,7 @@ import Loadable from 'react-loadable';
 
 import path from 'path';
 import fs from 'fs';
-import routes from '@generated/routes'; // eslint-disable-line
+import routes from '@generated/routes';
 import preload from './preload';
 import App from './App';
 import ssrTemplate from './templates/ssr.html.template';
diff --git a/packages/docusaurus/src/client/theme-fallback/Layout/index.js b/packages/docusaurus/src/client/theme-fallback/Layout/index.js
index fdcb856e1b..a9a9639aeb 100644
--- a/packages/docusaurus/src/client/theme-fallback/Layout/index.js
+++ b/packages/docusaurus/src/client/theme-fallback/Layout/index.js
@@ -6,8 +6,8 @@
  */
 
 import React from 'react';
-import Head from '@docusaurus/Head'; // eslint-disable-line
-import useDocusaurusContext from '@docusaurus/useDocusaurusContext'; // eslint-disable-line
+import Head from '@docusaurus/Head';
+import useDocusaurusContext from '@docusaurus/useDocusaurusContext';
 
 function Layout(props) {
   const context = useDocusaurusContext();
diff --git a/website/pages/index.js b/website/pages/index.js
index 63659df4fe..b9a6e60327 100644
--- a/website/pages/index.js
+++ b/website/pages/index.js
@@ -125,6 +125,7 @@ function Home() {
             <div className="col">
               <img
                 className={styles.featureImage}
+                alt={'Powered by Markdown'}
                 src={`${siteConfig.baseUrl}img/undraw_typewriter.svg`}
               />
               <h3>Powered by Markdown</h3>
@@ -136,6 +137,7 @@ function Home() {
             </div>
             <div className="col">
               <img
+                alt={'Built Using React'}
                 className={styles.featureImage}
                 src={`${siteConfig.baseUrl}img/undraw_react.svg`}
               />
@@ -148,6 +150,7 @@ function Home() {
             </div>
             <div className="col">
               <img
+                alt={'Ready for Translations'}
                 className={styles.featureImage}
                 src={`${siteConfig.baseUrl}img/undraw_around_the_world.svg`}
               />
@@ -163,6 +166,7 @@ function Home() {
           <div className="row">
             <div className="col col--4 col--offset-2">
               <img
+                alt={'Document Versioning'}
                 className={styles.featureImage}
                 src={`${siteConfig.baseUrl}img/undraw_version_control.svg`}
               />
@@ -175,6 +179,7 @@ function Home() {
             </div>
             <div className="col col--4">
               <img
+                alt={'Document Search'}
                 className={styles.featureImage}
                 src={`${siteConfig.baseUrl}img/undraw_algolia.svg`}
               />