Skip to content

fix(ci): qodana-main-fail durch redundante usings beheben#114

Merged
tomtastisch merged 2 commits intomainfrom
codex/fix/main-qodana-redundant-usings
Feb 21, 2026
Merged

fix(ci): qodana-main-fail durch redundante usings beheben#114
tomtastisch merged 2 commits intomainfrom
codex/fix/main-qodana-redundant-usings

Conversation

@tomtastisch
Copy link
Owner

@tomtastisch tomtastisch commented Feb 21, 2026

Ziel & Scope

Umgesetzte Aufgaben (abhaken)

  • Root-Cause aus fehlgeschlagenen Main-Runs identifiziert (qodana High+ Blocking Findings).
  • Minimaler Codefix umgesetzt (2 redundante using entfernt).
  • Lokale Verifikation (Build/Test) erfolgreich ausgefuehrt.

Nachbesserungen aus Review (iterativ)

  • Alle Copilot-/Reviewer-Kommentare geprueft
  • Alle notwendigen Code-Nachbesserungen umgesetzt
  • PR-Beschreibung nach Nachbesserung aktualisiert
  • Alle Threads resolved (inkl. outdated)

Security- und Merge-Gates

  • Required Checks sind gruen
  • security/code-scanning/tools: 0 offene Alerts (wird nach Abschluss aller Checks final verifiziert)
  • Keine offenen blocker findings im Diff (nur redundante usings entfernt)

Evidence (auditierbar)

  • Build/Test/Checks mit Artefakten vorhanden
  • Relevante Befehle im PR-Text dokumentiert
  • Risiken/Assumptions explizit benannt

Befehle/Evidence:

  • gh run view 22262987588 --json ... -> CI-QODANA-004 wegen High+ Blocking Findings.
  • jq ... /tmp/run-22262987588/ci-qodana/report/results/result-allProblems.json -> 2 verbleibende High-Funde RedundantUsingDirective.
  • dotnet build src/FileTypeDetection/FileTypeDetectionLib.vbproj -c Release --no-restore -v minimal -> Erfolg, 0 Warnungen, 0 Fehler.
  • dotnet test tests/FileTypeDetectionLib.Tests/FileTypeDetectionLib.Tests.csproj -c Release --no-build -v minimal -> Erfolg, 551/551.

DoD (mindestens 2 pro Punkt)

Punkt DoD A DoD B Status
Root-Cause Fehlrun-ID dokumentiert (22262987588) Blocking Rule (CI-QODANA-004) mit Evidence belegt [x]
Codefix Nur 2 Zeilen geloescht, keine Logik geaendert Betroffene Dateien exakt benannt [x]
Verifikation Build lokal gruen Tests lokal gruen [x]

Risiken / Open Items

  • keine
  • vorhanden (unten auflisten)

Details (nur falls vorhanden)

  • Risiko 1: Endgueltiger Nachweis der Behebung erfolgt ueber PR-CI (Qodana/Preflight) nach Push.

Copilot AI review requested due to automatic review settings February 21, 2026 19:41
@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@github-actions github-actions bot added area:tests fix Bugfix versioning:patch Fix/Refactor/Docs/CI/Tooling; requires PATCH bump impl:quality labels Feb 21, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Behebt eine Main-Regression im CI, indem zwei von Qodana als RedundantUsingDirective geflaggte using-Direktiven aus Unit-Tests entfernt werden (kein Produktionscode betroffen).

Changes:

  • Entfernt using Tomtastisch.FileClassifier; aus StreamGuardUnitTests.
  • Entfernt using Tomtastisch.FileClassifier; aus LogGuardUnitTests.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
tests/FileTypeDetectionLib.Tests/Unit/StreamGuardUnitTests.cs Entfernt ein ungenutztes using, um Qodana-Finding zu beseitigen.
tests/FileTypeDetectionLib.Tests/Unit/LogGuardUnitTests.cs Entfernt ein ungenutztes using, um Qodana-Finding zu beseitigen.

@tomtastisch tomtastisch merged commit 988a990 into main Feb 21, 2026
26 checks passed
@tomtastisch tomtastisch deleted the codex/fix/main-qodana-redundant-usings branch February 21, 2026 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:tests fix Bugfix impl:quality versioning:patch Fix/Refactor/Docs/CI/Tooling; requires PATCH bump

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants