Skip to content

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

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

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

Conversation

@ringo380

Copy link
Copy Markdown
Owner

Summary

Auto-generated by the QueryGrade weekly lint routine (2026-06-15).

Files changed: 3 files changed, 8 insertions(+), 4 deletions(-)

  • analyzer/urls.py — import order fixed by isort
  • analyzer/views/ml_alert_views.py — import order fixed by isort
  • analyzer/ml/tests/test_alert_notifier.py — import order fixed by isort

black: no changes (207 files already conformant)
isort: 3 files reformatted


Outstanding flake8 findings (manual fix required)

1182 findings across analyzer/ and querygrade/. Breakdown by error code:

Code Count Description
E501 744 Line too long (>88 characters)
F401 332 Imported but unused
F841 51 Local variable assigned but never used
F541 19 f-string without placeholders
F405 14 May be undefined or from star imports
E402 8 Module level import not at top of file
F811 7 Redefinition of unused name from import
F403 5 Star import used
F821 2 Undefined name
First 80 findings (click to expand)
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)
analyzer/analyzers/database/postgresql_analyzer.py:65:89: E501 line too long (114 > 88 characters)
analyzer/analyzers/database/postgresql_analyzer.py:78:89: E501 line too long (99 > 88 characters)
analyzer/analyzers/database/postgresql_analyzer.py:91:89: E501 line too long (108 > 88 characters)
analyzer/analyzers/database/postgresql_analyzer.py:104:89: E501 line too long (120 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:69:89: E501 line too long (97 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:86:89: E501 line too long (124 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:87:89: E501 line too long (102 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:113:89: E501 line too long (140 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:120:89: E501 line too long (126 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:121:89: E501 line too long (155 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:132:89: E501 line too long (90 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:139:89: E501 line too long (95 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:140:89: E501 line too long (106 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:160:89: E501 line too long (137 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:167:89: E501 line too long (129 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:168:89: E501 line too long (121 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:182:89: E501 line too long (91 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:193:89: E501 line too long (140 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:200:89: E501 line too long (126 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:201:89: E501 line too long (105 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:206:89: E501 line too long (99 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:216:89: E501 line too long (121 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:217:89: E501 line too long (108 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:231:89: E501 line too long (117 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:232:89: E501 line too long (101 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:245:89: E501 line too long (114 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:256:89: E501 line too long (94 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:257:89: E501 line too long (113 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:267:89: E501 line too long (114 > 88 characters)
analyzer/analyzers/groupby_analyzer.py:274:89: E501 line too long (119 > 88 characters)

(Full output: 1182 lines — see CI artifact or run flake8 analyzer/ querygrade/ locally)


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 and unrelated to this PR.

The Test Suite reports 61 errors, all with the same root cause:

ValueError: Missing staticfiles manifest entry for 'analyzer/img/favicon.svg'

This is a test-environment setup issue — collectstatic isn't being run before the test suite, so ManifestStaticFilesStorage can't resolve hashed asset URLs. Our changes are purely isort import reordering across 3 files; they cannot cause a staticfiles manifest to be missing.

The last commit to main (#93, 2026-05-21) noted "Full analyzer suite green", so this CI breakage appeared sometime after that merge and is independent of this lint sweep.


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