Merge pull request #688 from OWASP/feat/add-eslint

feat: add eslint to pre-commit

Author: commjoen

.eslintrc.js

@@ -0,0 +1,15 @@
+module.exports = {
+  env: {
+    browser: true,
+    commonjs: true,
+    es2021: true
+  },
+  extends: 'standard',
+  overrides: [
+  ],
+  parserOptions: {
+    ecmaVersion: 'latest'
+  },
+  rules: {
+  }
+}

.github/workflows/pre-commit.yml

@@ -23,6 +23,12 @@ jobs:
         uses: actions/setup-python@v4
         with:
           python-version: "3.9"
+      - uses: actions/setup-node@v3
+        with:
+          node-version: 18
+          cache: "npm"
+      - name: Install npm dependencies
+        run: npm install
       - uses: actions/cache@v3
         name: Cache plugin dir
         with:

.pre-commit-config.yaml

@@ -32,3 +32,9 @@ repos:
     hooks:
       - id: commitlint
         stages: [commit-msg]
+  - repo: https://github.com/eslint/eslint
+    rev: v8.35.0
+    hooks:
+      - id: eslint
+        args:
+          - "--fix"

commitlint.config.js

@@ -1,6 +1,6 @@
 // commitlint.config.js
 module.exports = {
-    extends: [
-        "@commitlint/config-conventional", // scoped packages are not prefixed
-    ],
-};
+  extends: [
+    '@commitlint/config-conventional' // scoped packages are not prefixed
+  ]
+}

js/index.js

@@ -1,4 +1,5 @@
-function secret() {
- var password = "jLJIQZM=" + 9 + "aG7l" + 6 + "LAQ=" + 2 + "UdiM" + 7;
- return password;
- }
+/* eslint-disable no-unused-vars */
+function secret () {
+  const password = 'jLJIQZM=' + 9 + 'aG7l' + 6 + 'LAQ=' + 2 + 'UdiM' + 7
+  return password
+}