Fix Precommit (#279)

* 🔥 Add validate files * Update pre-commit to actually work * Fixed pre-commit-config finally * Lock * Update pip * Pipfile lock * Update README --------- Co-authored-by: Joy Liu <[email protected]> Co-authored-by: Justin Zhang <[email protected]>
pennlabs · Apr 14, 2024 · 39ee027 · 39ee027
1 parent f7220cf
commit 39ee027
Show file tree

Hide file tree

Showing 5 changed files with 85 additions and 8 deletions.
diff --git a/.github/workflows/validate.yaml b/.github/workflows/validate.yaml
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -1,10 +1,11 @@
 repos:
   - repo: https://github.com/pennlabs/pre-commit-hooks
-    rev: stable
+    rev: v1.0-mobile-backend
     hooks:
-        - id: isort
-        - id: flake8
-  - repo: https://github.com/psf/black
-    rev: 22.3.0
-    hooks:
-        - id: black
+      - id: black
+        args: [-l100]
+      - id: isort
+        args: []
+      - id: flake8
+        args: [--config, backend/setup.cfg]
+      - id: detect-private-key
diff --git a/README.md b/README.md
@@ -28,6 +28,9 @@ Final Steps:
 - `pipenv run python manage.py migrate`
 - `pipenv run python manage.py runserver 8000`
 
+Setting up precommit:
+- `pipenv run pre-commit install`
+
 ## Creating Users
 
 To create users, you first have to create a main superuser.

diff --git a/backend/Pipfile b/backend/Pipfile
@@ -46,6 +46,7 @@ redis = "*"
 python-dateutil = "*"
 selenium = "*"
 webdriver-manager = "*"
+pre-commit = "*"
 
 [requires]
 python_version = "3.11"
diff --git a/backend/Pipfile.lock b/backend/Pipfile.lock