Merge pull request #75 from effekt-lang/linter-formatter

Add ESLint and Prettier as linter and formatter

Author: timsueberkrueb

.git-blame-ignore-revs

@@ -0,0 +1,11 @@
+# .git-blame-ignore-revs
+
+# using `git blame --ignore-revs-file .git-blame-ignore-revs`
+# or configuring git to always do so : `git config blame.ignoreRevsFile .git-blame-ignore-revs`
+
+
+# First time linting and formatting the whole codebase
+87e78e4db73d7f28aad21613bc64ed1ea80c4d6d
+
+# Reformat with trailing commata on 
+0c41364313629aa1cf42c0b1f40c3f78c7e48196

.github/workflows/ci.yml

@@ -21,5 +21,11 @@ jobs:
     - run: npm install -g vsce
     - run: npm install
 
+    - name: Run ESLint
+      run: npm run lint
+
+    - name: Run Prettier Check
+      run: npm run format:check
+      
     - name: Package VSCode extension into .vsix file
       run: vsce package

.prettierrc

@@ -0,0 +1,6 @@
+{
+  "semi": true,
+  "trailingComma": "all",
+  "singleQuote": true,
+  "printWidth": 80
+}

eslint.config.mjs

@@ -0,0 +1,48 @@
+/**
+ * ESLint configuration for the project.
+ * 
+ * See https://eslint.style and https://typescript-eslint.io for additional linting options.
+ */
+// @ts-check
+import js from '@eslint/js';
+import tseslint from 'typescript-eslint';
+import prettier from 'eslint-config-prettier';
+import prettierPlugin from 'eslint-plugin-prettier';
+
+export default tseslint.config(
+    {
+        ignores: [
+            '**/.vscode-test',
+            '**/out',
+        ]
+    },
+    js.configs.recommended,
+    ...tseslint.configs.recommended,
+    ...tseslint.configs.stylistic,
+    prettier, // Disable conflicting ESLint rules
+    {
+        plugins: {
+            prettier: prettierPlugin // Enable Prettier plugin
+        },
+        rules: {
+            'prettier/prettier': 'error', // Treat Prettier issues as ESLint errors
+            'curly': 'warn',
+            '@typescript-eslint/no-empty-function': 'off',
+            '@typescript-eslint/naming-convention': [
+                'warn',
+                {
+                    'selector': 'import',
+                    'format': ['camelCase', 'PascalCase']
+                }
+            ],
+            '@typescript-eslint/no-unused-vars': [
+                'error',
+                {
+                    'argsIgnorePattern': '^_', // Allow unused variables with a prefix '_'
+                    'varsIgnorePattern': '^_'  // Allow unused vars with a prefix '_'
+                }
+            ],
+            '@typescript-eslint/no-explicit-any': 'off' // Allow 'any' type
+        }
+    }
+);