refactor: linting tweaks (#2507)

Flagsmith · Jul 31, 2023 · 35a4f06 · 35a4f06 · vercel · Jul 31, 2023
1 parent 591bd07
commit 35a4f06
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 5 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -4,23 +4,30 @@ repos:
     hooks:
       - id: isort
         name: isort (python)
+
   - repo: https://github.com/psf/black
-    rev: 23.1.0
+    rev: 23.7.0
     hooks:
       - id: black
         language_version: python3
         exclude: migrations
+
   - repo: https://github.com/pycqa/flake8
     rev: 6.0.0
     hooks:
       - id: flake8
         name: flake8
         args: [--config, api/.flake8]
+
   - repo: https://github.com/pre-commit/pre-commit-hooks
     rev: v4.4.0
     hooks:
       - id: check-yaml
+      - id: check-json
+      - id: check-toml
+
   - repo: https://github.com/pre-commit/mirrors-prettier
     rev: v3.0.0
     hooks:
       - id: prettier
+        exclude: ^(frontend/|CHANGELOG.md)
diff --git a/api/.flake8 b/api/.flake8
@@ -4,6 +4,7 @@ max-complexity = 10
 exclude = .git,.venv,.direnv,__pycache__,*/migrations/*
 
 per-file-ignores =
-    app/settings/local.py: F403, F405
-    app/settings/production.py: F403, F405
-    app/settings/saas.py: F403, F405
+    # Need the * prefix to work with pre-commit which runs from the root of the repo
+    *app/settings/local.py: F403, F405
+    *app/settings/production.py: F403, F405
+    *app/settings/saas.py: F403, F405
diff --git a/frontend/tsconfig.json b/frontend/tsconfig.json
@@ -29,7 +29,7 @@
     }
   },
   "exclude": [
-    "node_modules",
+    "node_modules"
   ],
   "include": [
     "global.d.ts",