From 7906321ed3605a833c4cd4a10b0fdca69ad14bd3 Mon Sep 17 00:00:00 2001
From: JounQin <admin@1stg.me>
Date: Fri, 1 Nov 2019 22:47:20 +0800
Subject: [PATCH] fix(eslint-config): remove isSrcAppDirAvailable which seems
 useless

---
 packages/eslint-config/_util.js     | 12 ++++--------
 packages/eslint-config/index.js     |  2 +-
 packages/eslint-config/overrides.js | 10 ++--------
 3 files changed, 7 insertions(+), 17 deletions(-)

diff --git a/packages/eslint-config/_util.js b/packages/eslint-config/_util.js
index c968d582..a1c49e1a 100644
--- a/packages/eslint-config/_util.js
+++ b/packages/eslint-config/_util.js
@@ -1,8 +1,7 @@
-/* eslint-disable node/no-extraneous-require */
 const fs = require('fs')
 const { resolve } = require('path')
 
-const { tryFile, tryPkg } = require('@pkgr/utils')
+const { isPkgAvailable, tryFile } = require('@pkgr/utils')
 const isGlob = require('is-glob')
 const globSync = require('tiny-glob/sync')
 
@@ -53,11 +52,7 @@ try {
   exports.isSrcDirAvailable = fs.statSync(resolve('src')).isDirectory()
 } catch (e) {}
 
-try {
-  exports.isSrcAppDirAvailable = fs.statSync(resolve('src/app')).isDirectory()
-} catch (e) {}
-
-exports.isWebpackAvailable = tryPkg('webpack')
+exports.isWebpackAvailable = isPkgAvailable('webpack')
 
 // https://webpack.js.org/api/module-variables/#__resourcequery-webpack-specific
 exports.webpackSpecVars = [
@@ -76,7 +71,7 @@ exports.camelCaseRule = [
   {
     properties: 'never',
     ignoreDestructuring: true,
-    allow: exports.isWebpackAvailable && exports.webpackSpecVars,
+    allow: exports.isWebpackAvailable ? exports.webpackSpecVars : undefined,
   },
 ]
 
@@ -98,6 +93,7 @@ exports.magicNumbers = [
   100,
   365,
   500,
+  768,
   1000,
   1024,
   3600,
diff --git a/packages/eslint-config/index.js b/packages/eslint-config/index.js
index 06b63c04..c2f59362 100644
--- a/packages/eslint-config/index.js
+++ b/packages/eslint-config/index.js
@@ -38,7 +38,7 @@ module.exports = {
       ],
     },
   },
-  globals: isWebpackAvailable && getGlobals(webpackSpecVars),
+  globals: isWebpackAvailable ? getGlobals(webpackSpecVars) : undefined,
   rules: {
     camelcase: camelCaseRule,
     'import/order': [
diff --git a/packages/eslint-config/overrides.js b/packages/eslint-config/overrides.js
index d9602cd4..72b07f6f 100644
--- a/packages/eslint-config/overrides.js
+++ b/packages/eslint-config/overrides.js
@@ -10,12 +10,7 @@ const {
   tryPkg,
 } = require('@pkgr/utils')
 
-const {
-  camelCaseRule,
-  isSrcDirAvailable,
-  isSrcAppDirAvailable,
-  magicNumbers,
-} = require('./_util')
+const { camelCaseRule, isSrcDirAvailable, magicNumbers } = require('./_util')
 
 const configFile =
   tryFile(resolve('babel.config.js')) ||
@@ -75,8 +70,7 @@ const resolveSettings = {
     resolvePaths: [
       tryFile('node_modules/@d-ts', true),
       tryFile('node_modules/@types', true),
-      isSrcDirAvailable && !isAngularAvailable && 'src',
-      isAngularAvailable && isSrcAppDirAvailable && 'src/app',
+      isSrcDirAvailable && 'src',
     ].filter(Boolean),
     tryExtensions: [
       '.ts',