Repository navigation
fix(lint): remove unused imports/variable failing ruff CI on master - #32
Conversation
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01THB2Eyiff25dwsJcavXApb
|
Review (Zweitmodell): Diff ist minimal (3 Dateien, nur ungenutzte Imports/Variable entfernt), alle Test-Jobs und CodeQL grün. Aus meiner Sicht freigabefähig. Danach bitte #31 auf master rebasen – dessen Test-Fehler sind die F401-Ruff-Fehler ( Generated by Claude Code |
lukisch
left a comment
lukisch
left a comment
There was a problem hiding this comment.
Zweitprüfung: freigabefähig. Reine Entfernung ungenutzter Imports/Variable (Ruff), kein Verhaltenseffekt; alle 9 Test-Matrixjobs + CodeQL grün.
Generated by Claude Code
|
Review (Zweitmodell): Diff geprüft. Es werden nur ungenutzte Imports und eine ungenutzte Variable entfernt, das Verhalten ändert sich nicht. CI ist komplett grün, der Merge-Status ist clean. Aus meiner Sicht kann der PR gemergt werden. Danach #31 neu starten oder rebasen, denn dessen rote Tests sind dieselben ruff-F401-Fehler von master. Generated by Claude Code |
lukisch
left a comment
lukisch
left a comment
There was a problem hiding this comment.
Zweitprüfung: OK. Diff entfernt nur ungenutzte Imports/Variable (archive_dialog.py, 2 Tests), kein Verhaltenseffekt. Alle 14 Checks grün (3 OS × 3 Python, CodeQL). Merge-bereit.
Generated by Claude Code
lukisch
left a comment
lukisch
left a comment
There was a problem hiding this comment.
Zweitprüfung: Diff gelesen (3 Dateien, +3/−9). Es werden nur ungenutzte Imports und eine ungenutzte Variable (as tf) entfernt, das Laufzeitverhalten bleibt gleich. Alle 9 Testmatrix-Jobs (3.10–3.12 × Linux/macOS/Windows), CodeQL und Analyze sind grün, der Merge-Status ist clean. Aus meiner Sicht freigabefähig. Das Mergen bleibt beim Maintainer.
Generated by Claude Code
|
Review (Zweitmodell, nur Kommentar – keine Freigabe/Merge): Reine Lint-Bereinigung (ungenutzte Imports/Variable in Generated by Claude Code |
lukisch
left a comment
lukisch
left a comment
There was a problem hiding this comment.
Second-model review: LGTM. Only removes unused imports/variable (inspect_zip, is_archive are not referenced in archive_dialog.py; test changes are import-only plus an unused as tf). No behavior change; full CI matrix and CodeQL green. Safe to merge. (Formal approve not possible: same account as author.)
Generated by Claude Code
lukisch
left a comment
lukisch
left a comment
There was a problem hiding this comment.
Zweitprüfung (Cloud-Bot): Der Diff entfernt nur ungenutzte Imports und eine ungenutzte Variable. ruff check src tests ist auf dem PR-Branch grün. Die entfernten Namen (inspect_zip, is_archive, os, QApplication und weitere) werden in den betroffenen Dateien nicht mehr verwendet, und tests/test_archive_suite.py importiert sie nicht aus archive_dialog. Die Qt-Tests konnte ich in dieser Umgebung nicht ausführen, weil PySide6 fehlt. Der Fix ist unkritisch, die CI-Matrix sollte ihn bestätigen.
Generated by Claude Code
lukisch
left a comment
lukisch
left a comment
There was a problem hiding this comment.
Zweitprüfung (freigabefähig): Reine Entfernung ungenutzter Imports/Variable, kein Verhaltenswechsel. Alle 9 Test-Matrixjobs und CodeQL grün.
Generated by Claude Code
Behebt
ruff check .(F401 x9, F841 x1) insrc/gui/archive_dialog.py,tests/test_archive_gui_integration.py,tests/test_tar_archive_suite.py. Master-CI und damit PR #31 (docs-only) waren deswegen rot. Nur Entfernen ungenutzter Imports/Variablen, kein Verhaltenswechsel;ruff check .lokal grün.Generated by Claude Code