Merge pull request #534 from gselderslaghs/eslint-coding-standards

Eslint coding standards

eslint.config.mjs

@@ -1,6 +1,4 @@
 import typescriptEslint from 'typescript-eslint';
-import prettierPlugin from 'eslint-plugin-prettier';
-import eslintConfigPrettier from 'eslint-config-prettier';
 
 export default [
   {
@@ -9,27 +7,17 @@ export default [
   },
   ...typescriptEslint.configs.recommended,
   {
-    plugins: {
-      prettier: prettierPlugin
-    },
     rules: {
-      ...prettierPlugin.configs.recommended.rules,
-      ...eslintConfigPrettier.rules,
-
-      // TODO: to improve because the basic rules cause
-      // thousands of errors, for this reason, they have been marked as WARN
-      'prettier/prettier': 'warn',
-
-      'prefer-const': 'warn',
+      'prefer-const': 'error',
       'prefer-rest-params': 'warn',
       'no-var': 'warn',
-      '@typescript-eslint/no-explicit-any': 'warn',
-      '@typescript-eslint/no-unused-vars': 'warn',
-      '@typescript-eslint/no-unused-expressions': 'warn',
+      '@typescript-eslint/no-explicit-any': 'error',
+      '@typescript-eslint/no-unused-vars': 'error',
+      '@typescript-eslint/no-unused-expressions': 'error',
       '@typescript-eslint/no-this-alias': 'warn',
-      '@typescript-eslint/no-empty-object-type': 'warn',
-      '@typescript-eslint/no-require-imports': 'warn',
-      '@typescript-eslint/no-unsafe-function-type': 'warn'
+      '@typescript-eslint/no-empty-object-type': 'error',
+      '@typescript-eslint/no-require-imports': 'error',
+      '@typescript-eslint/no-unsafe-function-type': 'error'
     }
   }
 ];

package.json

@@ -58,12 +58,9 @@
     "@types/node": "^20.5.2",
     "archiver": "^7.0.1",
     "eslint": "^9.10.0",
-    "eslint-config-prettier": "^9.1.0",
-    "eslint-plugin-prettier": "^5.2.1",
     "jasmine": "^5.1.0",
     "jasmine-browser-runner": "^2.5.0",
     "jasmine-core": "^5.2.0",
-    "prettier": "^3.3.3",
     "rollup": "^4.20.0",
     "rollup-plugin-copy": "^3.5.0",
     "rollup-plugin-dts": "^6.1.1",