Skip to content

ci: install pytest in test workflow jobs - #3

Merged
lukisch merged 1 commit into
masterfrom
claude/practical-cori-f2io7m
Oct 4, 2026
Merged

lukisch merged 1 commit into
masterfrom
claude/practical-cori-f2io7m

Conversation

@lukisch

@lukisch lukisch commented Oct 3, 2026

Copy link
Copy Markdown
Collaborator

Zusammenfassung

Die Test-Jobs (Windows, Linux-Smoke, macOS-Smoke) installieren nur requirements.txt; pytest fehlte. Es wird jetzt in allen drei Install-Schritten mit installiert.

Test

  • Nicht lokal ausgeführt (Workflow-only, Windows-/Qt-Umgebung nötig); CI prüft.

🤖 Generated with Claude Code

https://claude.ai/code/session_01VnxPUjm2uYvYzB7PEC3ZFL


Generated by Claude Code

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VnxPUjm2uYvYzB7PEC3ZFL
@github-actions

github-actions Bot commented Oct 3, 2026

Copy link
Copy Markdown

Welcome! 👋 Thanks for your first pull request in this repository.

A maintainer will review it soon. Please make sure:

  • Your changes are tested
  • Documentation is updated if needed
  • The PR description explains what and why

Thanks for contributing!

lukisch commented Oct 3, 2026

Copy link
Copy Markdown
Collaborator Author

Zweitprüfung (Cloud-Bot): Diff minimal (pytest in den drei Test-Jobs nachinstalliert), alle 10 Checks grün. Keine Beanstandung, merge-bereit. (Formale Approval nicht möglich: gleicher Account.)


Generated by Claude Code

@lukisch lukisch left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Zweitmodell-Review: Fügt pytest zu den drei Dependency-Install-Schritten hinzu (CI-Fix). Alle Checks grün (test 3.11/3.12, lint, Plattform-Smokes, CodeQL). Optional: requirements.txt bzw. ein dev-Extra wäre der dauerhaftere Ort. Freigabe empfohlen.


Generated by Claude Code

lukisch commented Oct 4, 2026

Copy link
Copy Markdown
Collaborator Author

Zweitprüfung (Cloud-Bot): ✅ unbedenklich. pytest in den drei Test-Jobs nachinstalliert, sonst nichts. Alle 10 Checks grün. Merge-bereit.


Generated by Claude Code

@lukisch lukisch left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Zweitmodell-Review: ✅ Freigabeempfehlung. pytest wird in allen drei Jobs installiert, alle 10 Checks sind grün. (Formale Approval ist nicht möglich, weil der Token dem PR-Autor gehört.)


Generated by Claude Code

@lukisch lukisch left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Zweitmodell-Review: pytest wird in allen drei Test-Jobs nachinstalliert; alle 10 Checks (test 3.11/3.12, lint, Platform-Smokes, CodeQL) grün. Hinweis: Langfristig könnte pytest in eine requirements-dev.txt/Extras-Gruppe wandern, statt in drei Workflow-Zeilen dupliziert zu sein – kein Blocker. Freigabefähig.


Generated by Claude Code

@lukisch lukisch left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Zweitmodell-Review: Minimaler CI-Fix (pytest in den drei Test-Jobs installiert), alle Checks grün. Freigabefähig.


Generated by Claude Code

lukisch commented Oct 4, 2026

Copy link
Copy Markdown
Collaborator Author

Zweitmodell-Review: ✅ freigabereif. CI grün (test 3.11/3.12, lint, linux/macos smoke, CodeQL). pytest in allen drei Jobs – minimal und korrekt.


Generated by Claude Code

@lukisch lukisch left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Zweitprüfung ✅ (Approve technisch nicht möglich, da eigener PR): pytest wird in den drei Test-Jobs installiert, sonst keine Änderung. Alle 10 Checks grün. Aus Reviewsicht mergebar.


Generated by Claude Code

@lukisch
lukisch merged commit 2b97917 into master Oct 4, 2026
10 checks passed
@lukisch
lukisch deleted the claude/practical-cori-f2io7m branch October 4, 2026 10:06
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