Skip to content

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

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

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

Conversation

@ringo380

@ringo380 ringo380 commented Sep 7, 2026

Copy link
Copy Markdown
Owner

Description

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

Summary: 15 files changed, 58 insertions(+), 56 deletions(-)

Type of Change

  • Refactoring (no functional changes)

Related Issues

N/A — routine automated maintenance

Changes Made

  • Applied black reformatting to 12 files
  • Applied isort import sorting to 4 files
  • Files: analyzer/management/commands/process_ml_feedback.py, train_ml_model.py, analyzer/migrations/0008_mlmodelartifact.py, analyzer/ml/core/training_gates.py, hybrid_grader.py, training_pipeline.py, analyzer/ml/tests/test_model_storage.py, test_process_ml_feedback_command.py, test_training_gates.py, test_alert_notifier.py, analyzer/test_analytics.py, analyzer/test_seo.py, analyzer/urls.py, analyzer/views/ml_alert_views.py, querygrade/settings.py

Testing

  • Unit tests pass (python manage.py test)
  • No functional changes — formatting only

Checklist

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

Outstanding flake8 findings (manual fix required)

There are 1172 flake8 findings remaining that require manual fixes (not auto-fixable by black/isort). The majority are E501 line-too-long violations. A sample:

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/analyzers/base.py:19:1: F401 'django.core.cache.caches' imported but unused
analyzer/analyzers/base.py:158:9: F841 local variable 'start_time' is assigned to but never used

Full flake8 output was captured during this run (1172 lines). Notable error codes:

  • E501 — line too long (majority of findings)
  • F401 — imported but unused
  • F841 — local variable assigned but never used
  • E402 — module level import not at top of file

Additional Notes

This PR was created automatically by the QueryGrade weekly lint routine (scheduled task). Do not merge without human review.

https://claude.ai/code/session_01FMugAAMTiVr5pib8ELQw1Z


Generated by Claude Code

Auto-generated by the QueryGrade weekly lint routine.
Tooling: black + isort across analyzer/ and querygrade/.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FMugAAMTiVr5pib8ELQw1Z

This branch has not been deployed

No deployments
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