@odg/eslint-config 1.17.0 → 1.19.0

package/README.md

@@ -518,6 +518,7 @@
   - [No Invariant Returns](#no-invariant-returns)
   - [Inconsistent Function Call](#inconsistent-function-call)
   - [Duplicate Conditions](#duplicate-conditions)
+  - [Redundant Type Aliases](#redundant-type-aliases)
   - [No Element Overwrite](#no-element-overwrite)
   - [No Empty Collection](#no-empty-collection)
   - [No Extra Arguments](#no-extra-arguments)
@@ -19821,7 +19822,34 @@ switch (i) { // Noncompliant
 }
 ```
 
-## No Element Overwrite
+### Redundant Type Aliases
+
+----------
+
+Redundant type aliases should not be used
+
+<https://sonarsource.github.io/rspec/#/rspec/S6564/javascript>
+
+👍 Examples of correct code
+
+```typescript
+function isPalindrom(s: string): boolean {
+    return s === s.split('').reverse().join('');
+}
+```
+
+👎 Examples of incorrect code
+
+```typescript
+type MyString  = string;
+type MyBoolean = boolean;
+
+function isPalindrom(s: MyString): MyBoolean {
+    return s === s.split('').reverse().join('');
+}
+```
+
+### No Element Overwrite
 
 ----------
 
@@ -19853,7 +19881,7 @@ mySet.add(1);
 mySet.add(1); // Noncompliant - element is already in the set
 ```
 
-## No Empty Collection
+### No Empty Collection
 
 ----------
 
@@ -19886,7 +19914,7 @@ for (str of strings) {}  // Noncompliant
 strings.forEach(str => doSomething(str)); // Noncompliant
 ```
 
-## No Extra Arguments
+### No Extra Arguments
 
 ----------
 
@@ -19923,7 +19951,7 @@ function say(a, b) {
 say("hello", "world", "!"); // Noncompliant; last argument is not used
 ```
 
-## No Identical Expressions
+### No Identical Expressions
 
 ----------
 
@@ -20014,7 +20042,7 @@ function foo() {
 a = foo();
 ```
 
-## No Collection Size Mischeck
+### No Collection Size Mischeck
 
 ----------
 
@@ -20045,7 +20073,7 @@ if (someMap.size < 0) {...} // Noncompliant
 const result = someArray.length >= 0;  // Noncompliant
 ```
 
-## No Gratuitous Expressions
+### No Gratuitous Expressions
 
 ----------
 

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@odg/eslint-config",
-    "version": "1.17.0",
+    "version": "1.19.0",
     "description": "Linter for JavaScript And Typescript project",
     "main": "index.js",
     "author": "Dragons Gamers <https://www.linkedin.com/in/victor-alves-odgodinho>",
@@ -18,6 +18,7 @@
         "typescript": "*"
     },
     "peerDependencies": {
+        "@typescript-eslint/parser": "*",
         "eslint": "*"
     },
     "dependencies": {
@@ -28,29 +29,29 @@
         "any-eslint-parser": "^1.0.1",
         "eslint": "*",
         "eslint-import-resolver-typescript": "^3.5.5",
-        "eslint-plugin-antfu": "^0.38.6",
+        "eslint-plugin-antfu": "^0.40.0",
         "eslint-plugin-array-func": "^3.1.8",
         "eslint-plugin-better-max-params": "^0.0.1",
         "eslint-plugin-file-progress": "^1.3.0",
         "eslint-plugin-filenames": "^1.3.2",
         "eslint-plugin-html": "^7.1.0",
-        "eslint-plugin-import": "^2.27.5",
-        "eslint-plugin-jsdoc": "^44.2.3",
-        "eslint-plugin-json-schema-validator": "^4.5.0",
-        "eslint-plugin-jsonc": "^2.8.0",
-        "eslint-plugin-n": "^15.7.0",
+        "eslint-plugin-import": "^2.28.0",
+        "eslint-plugin-jsdoc": "^46.4.5",
+        "eslint-plugin-json-schema-validator": "^4.6.0",
+        "eslint-plugin-jsonc": "^2.9.0",
+        "eslint-plugin-n": "^16.0.1",
         "eslint-plugin-no-constructor-bind": "^2.0.4",
         "eslint-plugin-php-markup": "^6.0.0",
         "eslint-plugin-promise": "^6.1.1",
         "eslint-plugin-regex": "^1.10.0",
         "eslint-plugin-regexp": "^1.15.0",
         "eslint-plugin-security": "^1.7.1",
-        "eslint-plugin-sonar": "^0.11.1",
+        "eslint-plugin-sonar": "^0.12.0",
         "eslint-plugin-sonarjs": "^0.19.0",
         "eslint-plugin-sort-class-members": "^1.18.0",
         "eslint-plugin-toml": "^0.5.0",
-        "eslint-plugin-unicorn": "^47.0.0",
-        "eslint-plugin-yml": "^1.7.0",
+        "eslint-plugin-unicorn": "^48.0.1",
+        "eslint-plugin-yml": "^1.8.0",
         "toml-eslint-parser": "^0.6.0"
     },
     "files": [

package/rules/global/possible-errors.js

@@ -68,6 +68,7 @@ module.exports = {
         "sonarjs/no-duplicated-branches": [ "error" ], // Em vez de if else igual faça um ||
         "sonarjs/no-gratuitous-expressions": [ "error" ], // Não faça a mesma condição dentro de outra
         "sonarjs/no-unused-collection": [ "error" ], // Não faça array que não é usado
+        "sonar/redundant-type-aliases": [ "error" ], // Não crie tipos redundantes
         "regexp/no-extra-lookaround-assertions": [ "error" ], // Look around invalido em regex
     },
 };

package/rules/javascript/best-practices.js

@@ -455,5 +455,7 @@ module.exports = {
         "antfu/generic-spacing": [ "error" ], // String<a,b> => String<a, b>
         "antfu/import-dedupe": [ "error" ], // Auto fix import duplicados
         "antfu/prefer-inline-type-import": [ "error" ], // Prefira type inline
+        "antfu/named-tuple-spacing": [ "error" ], // Solicita espaço depois do : no Typescript types
+        "antfu/no-import-node-modules-by-path": [ "error" ], // Não importe de dentro da node_modules,
     },
 };

package/rules/typescript/best-practices.js

@@ -477,6 +477,6 @@ module.exports = {
             },
         ], // Não permite numero mágicos
         "@typescript-eslint/prefer-for-of": "error",
-        "@typescript-eslint/restrict-plus-operands": [ "error", { "checkCompoundAssignments": true } ],
+        "@typescript-eslint/restrict-plus-operands": [ "error" ],
     },
 };

package/rules/typescript/possible-errors.js

@@ -27,5 +27,6 @@ module.exports = {
         "sonarjs/no-extra-arguments": [ "off" ], // Argumentos extra ja é tratado por typescript
         "sonarjs/no-use-of-empty-return-value": [ "off" ], // Não atribua void em uma variável
         "@odg/no-inconsistent-docblock": [ "error" ], // Valida Docblock @param e @return com tipagem
+        "antfu/no-const-enum": [ "error" ], // Não coloque enum em constant
     },
 };