Skip to content

chore(lint): weekly black/isort/flake8 sweep - #139

Draft
ringo380 wants to merge 1 commit into
mainfrom
chore/weekly-lint-20260727
Draft

chore(lint): weekly black/isort/flake8 sweep#139
ringo380 wants to merge 1 commit into
mainfrom
chore/weekly-lint-20260727

Conversation

@ringo380

Copy link
Copy Markdown
Owner

Description

Auto-generated weekly lint sweep using black + isort across analyzer/ and querygrade/. No functional changes — formatting and import ordering only.

Type of Change

  • Refactoring (no functional changes)

Related Issues

N/A — automated routine

Changes Made

  • Applied black auto-formatting to 12 files
  • Applied isort import ordering to 4 files
  • Total: 15 files changed, 58 insertions(+), 56 deletions(-)

Files touched:

  • analyzer/management/commands/process_ml_feedback.py
  • analyzer/management/commands/train_ml_model.py
  • analyzer/migrations/0008_mlmodelartifact.py
  • analyzer/ml/core/hybrid_grader.py
  • analyzer/ml/core/training_gates.py
  • analyzer/ml/core/training_pipeline.py
  • analyzer/ml/tests/test_alert_notifier.py
  • analyzer/ml/tests/test_model_storage.py
  • analyzer/ml/tests/test_process_ml_feedback_command.py
  • analyzer/ml/tests/test_training_gates.py
  • analyzer/test_analytics.py
  • analyzer/test_seo.py
  • analyzer/urls.py
  • analyzer/views/ml_alert_views.py
  • querygrade/settings.py

Testing

  • No functional changes; no tests required for pure formatting
  • Unit tests pass (python manage.py test)

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my own code
  • My changes generate no new warnings

Outstanding flake8 findings (manual fix required)

flake8 reported 1,172 findings across analyzer/ and querygrade/. These require manual attention and are not addressed by this PR. Common categories:

  • E501 — lines exceeding 88 characters (majority of findings)
  • F401 — unused imports
  • F841 — local variables assigned but never used
  • E402 — module-level imports not at top of file
  • W291/W293 — trailing whitespace (in comments/strings black doesn't touch)

Sample (first 20):

analyzer/admin/__init__.py:7:89: E501 line too long (94 > 88 characters)
analyzer/admin/__init__.py:9:89: E501 line too long (92 > 88 characters)
analyzer/admin/__init__.py:16:1: F401 '.ml_admin' imported but unused
analyzer/admin/__init__.py:16:1: F401 '.query_admin' imported but unused
analyzer/admin/__init__.py:16:1: F401 '.user_admin' imported but unused
analyzer/admin/__init__.py:23:1: E402 module level import not at top of file
analyzer/admin/__init__.py:32:1: E402 module level import not at top of file
analyzer/admin/__init__.py:33:1: E402 module level import not at top of file
analyzer/admin/ml_admin.py:250:89: E501 line too long (141 > 88 characters)
analyzer/admin/ml_admin.py:251:89: E501 line too long (110 > 88 characters)
analyzer/admin/ml_admin.py:252:89: E501 line too long (138 > 88 characters)
analyzer/admin/ml_admin.py:327:89: E501 line too long (92 > 88 characters)
analyzer/admin/user_admin.py:39:89: E501 line too long (92 > 88 characters)
analyzer/analyzers/base.py:19:1: F401 'django.core.cache.caches' imported but unused
analyzer/analyzers/base.py:86:89: E501 line too long (89 > 88 characters)
analyzer/analyzers/base.py:158:9: F841 local variable 'start_time' is assigned to but never used
analyzer/analyzers/base.py:214:89: E501 line too long (91 > 88 characters)
analyzer/analyzers/base.py:215:89: E501 line too long (97 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:69:89: E501 line too long (118 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:76:89: E501 line too long (131 > 88 characters)

Generated by Claude Code

Auto-generated by the QueryGrade weekly lint routine.
Tooling: black + isort across analyzer/ and querygrade/.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants