Skip to content

fix(ci): Clippy und Testbuild des CI-Batches wieder grün - #541

Merged
iret77 merged 1 commit into
mainfrom
fix/ci-batch-clippy-tests
Oct 10, 2026
Merged

iret77 merged 1 commit into
mainfrom
fix/ci-batch-clippy-tests

Conversation

@iret77

@iret77 iret77 commented Oct 10, 2026 •

Copy link
Copy Markdown
Collaborator

Worum es geht

Der CI-Batch auf main (4e3ab3cf5, Lauf 38043837990) war unter Linux rot. Die Ursachen kamen mit den Merges vom 06. bis 08.10. Der Lauf vom 08.10. war schon vorher am Guard-Schritt gescheitert und hatte diese Fehler deshalb nie erreicht. macOS cargo check ist inzwischen grün.

Job Fehler Herkunft
cargo clippy (default = zaplex bin) ungenutzte Importe SftpPane und PaneContent in workspace/view.rs #522
cargo clippy (default = zaplex bin) explicit_auto_deref (5×), redundant_closure, needless_borrows_for_generic_args und enum_variant_names in cockpit/panel.rs #529
cargo check -p warp --tests E0063: Dem Literal von CockpitModel in cockpit/model_tests.rs fehlen sechs Felder #525

Änderungen

  • Die beiden ungenutzten Importe sind entfernt.
  • Die Clippy-Vorschläge im Session-Baum sind wörtlich übernommen. is_focused kopiert nur Referenzen und ist daher Copy, also darf es in der Schleife als Wert übergeben werden.
  • CockpitPanelAction hat nur noch Toggle*-Varianten. Deshalb steht dort jetzt #[allow(clippy::enum_variant_names)], wie in tab.rs und vier weiteren Dateien.
  • Das Testliteral setzt die sechs fehlenden Felder genauso wie der echte Konstruktor.

Ein lokaler Build fand nicht statt. Ob alles grün ist, zeigt erst der nächste CI-Batch auf main. Clippy stoppte diesmal an der ersten fehlerhaften Crate, der Testbuild am ersten Typfehler. Dahinter können also weitere Befunde liegen.

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Der CI-Batch auf main scheiterte unter Linux an zwei Stellen, die seit
den Merges vom 06. bis 08.10. offen waren:

- workspace/view.rs: Die Importe `SftpPane` und `PaneContent` sind seit
  dem gemeinsamen Startmenü (#522) ungenutzt.
- cockpit/panel.rs: explizite Derefs, eine redundante Closure und eine
  unnötige Referenz im Session-Baum (#529). `CockpitPanelAction` trägt
  nur noch Toggle-Varianten; das erlaubt die Datei jetzt wie im übrigen
  Code per `#[allow(clippy::enum_variant_names)]`.
- cockpit/model_tests.rs: Das Testliteral von `CockpitModel` kennt die
  sechs Felder aus dem Aufmerksamkeits-Zähler (#525) noch nicht.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@iret77
iret77 merged commit 7982fdf into main Oct 10, 2026
1 check passed
@iret77
iret77 deleted the fix/ci-batch-clippy-tests branch October 10, 2026 10:33
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.

1 participant