Adding flake8 checks in linting checks for modified python files (pgmpy#2176)

* Adding flake8 checks for modified python files in PRs

* added flake8 config file for pre-commit and github actions to borrow from

* Move flake8 checks to the linting workflow

* Fixes branch name in flake8 checks

* Fix branch name

* Fetch the dev branhc

* Fix linting and cancel any running workflow if a new commit is pushed

---------

Co-authored-by: Ankur Ankan <ankurankan@gmail.com>

Author: De-funkd

.flake8

@@ -0,0 +1,4 @@
+[flake8]
+max-line-length = 120
+ignore = E231,E241,E222,E221,W503
+exclude = .git,__pycache__,docs/source/conf.py,old,build,dist

.github/workflows/ci.yml

@@ -1,5 +1,9 @@
 name: Build and Run Test Suite
 
+concurrency:
+  group: ${{ github.workflow }}-${{ github.head_ref || github.ref }}
+  cancel-in-progress: true
+
 on: [push, pull_request]
 
 jobs:

.github/workflows/lint.yml

@@ -13,6 +13,8 @@ jobs:
     steps:
       - name: Checkout code
         uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
 
       - name: Setup Python ${{ matrix.python-version }}
         uses: actions/setup-python@v5
@@ -23,7 +25,7 @@ jobs:
       - name: Install linting tools
         run: |
           pip install --upgrade pip
-          pip install black blackdoc
+          pip install black blackdoc flake8
 
       - name: Check code formatting
         run: |
@@ -34,3 +36,21 @@ jobs:
         run: |
           blackdoc --diff pgmpy
           blackdoc --check pgmpy
+
+      - name: Fetch dev branch
+        run: git fetch origin dev:dev
+
+      - name: Get modified Python files
+        id: files
+        run: |
+          FILES=$(git diff --name-only dev...HEAD | grep '\.py$' | paste -sd' ' -)
+          echo "files=$FILES" >> "$GITHUB_OUTPUT"
+
+      - name: Flake8
+        run: |
+          if [ -n "${{ steps.files.outputs.files }}" ]; then
+            echo "Running flake8 on: ${{ steps.files.outputs.files }}"
+            flake8 --config=.flake8 ${{ steps.files.outputs.files }}
+          else
+            echo "No Python files changed."
+          fi

.pre-commit-config.yaml

@@ -1,26 +1,25 @@
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.6.0
+    rev: v5.0.0
     hooks:
       - id: check-yaml
       - id: end-of-file-fixer
       - id: trailing-whitespace
   - repo: https://github.com/pycqa/isort
-    rev: 5.13.2
+    rev: 6.0.1
     hooks:
       - id: isort
         args: ["--profile", "black", "--filter-files"]
   - repo: https://github.com/psf/black
-    rev: 24.8.0
+    rev: 25.1.0
     hooks:
       - id: black
+  - repo: https://github.com/Pycqa/flake8
+    rev: 7.2.0
+    hooks:
+      - id: flake8
   - repo: https://github.com/keewis/blackdoc
     rev: v0.4.0
     hooks:
     - id: blackdoc
-      additional_dependencies: ["black==24.8.0"]
-  - repo: https://github.com/PyCQA/flake8
-    rev: 7.2.0
-    hooks:
-      - id: flake8
-        args: ["--max-line-length=120", "--ignore=E231,E241,E222,E221,W503"]
+      additional_dependencies: ["black==24.8.0"]