Merge pull request #111 from creativecommons/refine-pre-commit

update pre-commit to use pipenv and drop pre-commit ci

Author: TimidRobot

.github/workflows/static_analysis.yml

@@ -3,7 +3,6 @@ name: Static Analysis
 on:
   pull_request:
   push:
-    branches: [main]
   workflow_dispatch:
 
 jobs:
@@ -18,18 +17,18 @@ jobs:
       with:
         python-version: '3.11'
 
-    - name: Install Python dependencies
+    - name: Install pipenv
       run: |
         pip install --upgrade pip
-        pip install black 'black[jupyter]' flake8 isort
-        pip install pre-commit
+        pip install pipenv
 
     # https://github.com/actions/checkout
     - name: Checkout quantifying
       uses: actions/checkout@v4
-      with:
-        path: quantifying
+
+    - name: Install Python dependencies
+      # Install _only_ [dev-packages]
+      run: pipenv sync --categories dev-packages --system
 
     - name: pre-commit
       run: pre-commit run --show-diff-on-failure --color=always --all-files
-      working-directory: ./quantifying

.pre-commit-config.yaml

@@ -3,22 +3,10 @@
 default_language_version:
   python: python3.11
 
-ci:
-    autofix_commit_msg: |
-        [pre-commit.ci] auto fixes from pre-commit.com hooks
-
-        for more information, see https://pre-commit.ci
-    autofix_prs: true
-    autoupdate_branch: ''
-    autoupdate_commit_msg: '[pre-commit.ci] pre-commit autoupdate'
-    autoupdate_schedule: weekly
-    skip: []
-    submodules: false
-
 repos:
 
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.1.0
+    rev: v4.6.0
     hooks:
       - id: check-added-large-files
       - id: check-ast