feat: apply eslint rule v5.9.0 ~ v5.12.0

nhn · Apr 12, 2019 · fd596ed · fd596ed
1 parent e14b77b
commit fd596ed
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 19 deletions.
diff --git a/LICENSE b/LICENSE
@@ -1,6 +1,6 @@
 The MIT License
 
-Copyright (c) 2017 NHN Entertainment Corp.
+Copyright (c) 2017 NHN Corp.
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

diff --git a/README.md b/README.md
@@ -40,9 +40,9 @@ module.exports = {
 ```
 
 ## Learn more
-[JavaScript Style Guide](https://github.com/nhnent/fe.javascript/wiki)
+[JavaScript Style Guide](https://github.com/nhn/fe.javascript/wiki)
 
 [Configuring ESLint](http://eslint.org/docs/user-guide/configuring)
 
 ## License
-This software is licensed under the [MIT License](https://github.com/nhnent/tui.eslint.config/blob/master/LICENSE).
+This software is licensed under the [MIT License](https://github.com/nhn/tui.eslint.config/blob/master/LICENSE).
diff --git a/package.json b/package.json
@@ -1,27 +1,27 @@
 {
   "name": "eslint-config-tui",
-  "version": "2.1.0",
+  "version": "2.2.0",
   "description": "ESLint sharable config for TUI components",
   "main": "index.js",
   "scripts": {
     "test": "echo \"Error: no test specified\" && exit 1"
   },
   "peerDependencies": {
-    "eslint": "^5.8.0"
+    "eslint": "^5.12.0"
   },
   "repository": {
     "type": "git",
-    "url": "https://github.com/nhnent/tui.eslint.config.git"
+    "url": "https://github.com/nhn/tui.eslint.config.git"
   },
   "keywords": [
     "eslint",
     "eslint-config",
     "eslint.config",
-    "nhnent",
+    "nhn",
     "tui",
     "tui.eslint.config",
     "tui.eslint"
   ],
-  "author": "NHNEnt FE Development Lab <dl_javascript@nhnent.com>",
+  "author": "NHN. FE Development Lab <dl_javascript@nhn.com>",
   "license": "MIT"
 }
diff --git a/rules/best-practices.js b/rules/best-practices.js
@@ -55,6 +55,7 @@ module.exports = {
         'no-useless-concat': 2,
         'no-useless-escape': 2,
         'no-useless-return': 2,
+        'no-useless-catch': 2,
         'no-void': 2,
         'no-warning-comments': 1,
         'no-with': 2,

diff --git a/rules/errors.js b/rules/errors.js
@@ -33,13 +33,6 @@ module.exports = {
         'no-unexpected-multiline': 2,
         'no-unreachable': 2,
         'use-isnan': 2,
-        'valid-jsdoc': [2, {
-            'prefer': {
-                'return': 'returns'
-            },
-            'requireReturnDescription': false,
-            'requireReturn': false
-        }],
         'valid-typeof': 2
     }
 };
diff --git a/rules/style.js b/rules/style.js
@@ -64,13 +64,14 @@ module.exports = {
         'padded-blocks': [2, 'never'],
         'padding-line-between-statements': [2,
             { blankLine: "always", prev: "*", next: "return" },
-            { blankLine: "always", prev: "directive", next: "*" }, { blankLine: "any", prev: "directive", next: "directive" },
-            { blankLine: "any", prev: ["const", "let", "var"], next: "*"}, { blankLine: "any", prev: ["const", "let", "var"], next: ["const", "let", "var"]},
-            { blankLine: "always", prev: "iife", next: "*" }, { blankLine: "always", prev: "*", next: "iife" }
+            { blankLine: "always", prev: "directive", next: "*" },
+            { blankLine: "any", prev: "directive", next: "directive" },
+            { blankLine: "any", prev: ["const", "let", "var"], next: "*"},
+            { blankLine: "always", prev: "iife", next: "*" },
+            { blankLine: "always", prev: "*", next: "iife" }
         ],
         'quote-props': 0,
         'quotes': [2, 'single', {'avoidEscape': true, 'allowTemplateLiterals': true}],
-        'require-jsdoc': 1,
         'require-unicode-regexp': 0,
         'semi': 2,
         'semi-spacing': [2, {'before': false, 'after': true}],