Skip to content

Commit 56186bd

Browse files
committed
CHORE: Remove flake8 - not enforced, only Black is required
- Remove flake8 from pyproject.toml lint dependencies - Remove flake8 from requirements.txt - Remove flake8 step from lint-check.yml workflow - Remove .flake8 trigger from workflow paths - Delete .flake8 config file Flake8 was never enforced (continue-on-error: true), only Black is blocking.
1 parent 4129ff5 commit 56186bd

4 files changed

Lines changed: 2 additions & 33 deletions

File tree

.flake8

Lines changed: 0 additions & 19 deletions
This file was deleted.

.github/workflows/lint-check.yml

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ on:
1212
- '**.hpp'
1313
- '.github/workflows/lint-check.yml'
1414
- 'pyproject.toml'
15-
- '.flake8'
1615
- '.clang-format'
1716
push:
1817
branches:
@@ -39,7 +38,7 @@ jobs:
3938
- name: Install dependencies
4039
run: |
4140
python -m pip install --upgrade pip
42-
pip install black flake8 pylint autopep8
41+
pip install black pylint autopep8
4342
if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
4443
4544
- name: Check Python formatting with Black
@@ -51,15 +50,6 @@ jobs:
5150
}
5251
echo "::endgroup::"
5352
54-
- name: Lint with Flake8
55-
run: |
56-
echo "::group::Flake8 Linting"
57-
flake8 mssql_python/ tests/ --max-line-length=100 --extend-ignore=E203,W503,E501,E722,F401,F841,W293,W291,F541,F811,E402,E711,E712,E721,F821 --count --statistics || {
58-
echo "::warning::Flake8 found linting issues (informational only, not blocking)"
59-
}
60-
echo "::endgroup::"
61-
continue-on-error: true
62-
6353
- name: Lint with Pylint
6454
run: |
6555
echo "::group::Pylint Analysis"
@@ -159,7 +149,7 @@ jobs:
159149
echo "❌ **Python Formatting (Black):** FAILED - Please run Black formatter" >> $GITHUB_STEP_SUMMARY
160150
fi
161151
162-
echo "ℹ️ **Python Linting (Flake8, Pylint):** Informational only" >> $GITHUB_STEP_SUMMARY
152+
echo "ℹ️ **Python Linting (Pylint):** Informational only" >> $GITHUB_STEP_SUMMARY
163153
echo "ℹ️ **C++ Linting (clang-format, cpplint):** Informational only" >> $GITHUB_STEP_SUMMARY
164154
165155
echo "" >> $GITHUB_STEP_SUMMARY

pyproject.toml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,6 @@ dev = [
6262
lint = [
6363
"black",
6464
"autopep8",
65-
"flake8",
6665
"pylint",
6766
"cpplint",
6867
"mypy",

requirements.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ setuptools
1212
# Code formatting and linting
1313
black
1414
autopep8
15-
flake8
1615
pylint
1716
cpplint
1817
mypy

0 commit comments

Comments
 (0)