Skip to content

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

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

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

Conversation

@ringo380
Copy link
Copy Markdown
Owner

Summary

Auto-generated by the QueryGrade weekly lint routine (2026-05-25).

  • black: no changes (207 files already compliant)
  • isort: fixed import ordering in 3 files
  • Files changed: 3 files changed, 8 insertions(+), 4 deletions(-)

Files reformatted by isort

  • analyzer/urls.py
  • analyzer/views/ml_alert_views.py
  • analyzer/ml/tests/test_alert_notifier.py

Outstanding flake8 findings (manual fix required)

Total: 1,182 findings across analyzer/ and querygrade/. Predominant issue types:

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

Representative sample (first 50 lines):

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)
analyzer/analyzers/case_statement_analyzer.py:77:89: E501 line too long (154 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:89:89: E501 line too long (126 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:96:89: E501 line too long (111 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:97:89: E501 line too long (120 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:105:89: E501 line too long (150 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:116:89: E501 line too long (123 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:123:89: E501 line too long (92 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:124:89: E501 line too long (148 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:135:89: E501 line too long (98 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:142:89: E501 line too long (95 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:143:89: E501 line too long (150 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:159:89: E501 line too long (117 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:166:89: E501 line too long (122 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:167:89: E501 line too long (153 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:181:89: E501 line too long (111 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:182:89: E501 line too long (139 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:192:89: E501 line too long (100 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:193:89: E501 line too long (120 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:207:89: E501 line too long (103 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:215:89: E501 line too long (130 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:238:89: E501 line too long (123 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:239:89: E501 line too long (140 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:250:89: E501 line too long (110 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:264:89: E501 line too long (99 > 88 characters)
analyzer/analyzers/case_statement_analyzer.py:265:89: E501 line too long (95 > 88 characters)
analyzer/analyzers/database/mysql_analyzer.py:70:89: E501 line too long (118 > 88 characters)
analyzer/analyzers/database/mysql_analyzer.py:80:89: E501 line too long (97 > 88 characters)
analyzer/analyzers/database/mysql_analyzer.py:93:89: E501 line too long (103 > 88 characters)
analyzer/analyzers/database/mysql_analyzer.py:106:89: E501 line too long (134 > 88 characters)
analyzer/analyzers/database/postgresql_analyzer.py:52:89: E501 line too long (89 > 88 characters)

Full output is at /tmp/flake8.txt in the build environment. These require manual fixes (line wrapping, removing unused imports, etc.) and are out of scope for this automated sweep.


Generated by Claude Code

Auto-generated by the QueryGrade weekly lint routine.
Tooling: black + isort across analyzer/ and querygrade/.
@ringo380
Copy link
Copy Markdown
Owner Author

CI failure is pre-existing — not introduced by this PR

The Test Suite failure (60 errors) is identical on main before this branch was created:

main:   720 tests, FAILED (errors=60, skipped=14)
PR #94: 720 tests, FAILED (errors=60, skipped=14)

All 60 errors are ValueError: Missing staticfiles manifest entry for '...' — WhiteNoise's CompressedManifestStaticFilesStorage requires a staticfiles/staticfiles.json manifest produced by collectstatic, but the CI workflow never runs collectstatic before the test step.

This lint sweep (import reordering only) did not cause or worsen any test failures. The underlying CI issue is a pre-existing gap in the workflow that should be fixed separately (add a python manage.py collectstatic --noinput step before the test run, or override STORAGES to use StaticFilesStorage in test mode).


Generated by Claude Code

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