test(ci): Action-Pins in test_metadata versionsunabhängig prüfen by lukisch · Pull Request #7 · dev-bricks/DevCenter

lukisch · 2026-10-11T05:48:48Z

Die Dependabot-PRs #4, #5, #6 (setup-python 7, github-script 9, checkout 7) schlagen fehl, weil tests/test_metadata.py die alten Major-Versionen (checkout@v4, setup-python@v5, github-script@v7) hart prüft (Log: assert 'actions/checkout@v4' in ..., 1 failed / 255 passed).

Fix: Assertions per Regex @v\d+. Nach dem Merge Dependabot per ·@·d·ependabot r·ebase neu laufen lassen.

Lokal nur Syntaxcheck (pytest nicht installiert); gleicher Fix ist in ellmos-ai/skills#63 bereits grün.

🤖 Generated with Claude Code

https://claude.ai/code/session_01RNAApyPmfEHABhzRXciAEk

Generated by Claude Code

…rsions Dependabot bumps of actions/checkout, setup-python and github-script failed tests/test_metadata.py because it hard-coded the old majors. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01RNAApyPmfEHABhzRXciAEk

github-actions · 2026-10-11T05:48:56Z

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 · 2026-10-11T06:48:08Z

Zweitprüfung (Cloud-Bot): freigabefähig. Der Diff ist minimal und korrekt (Regex @v\d+, import re ergänzt) und alle 10 Checks sind grün. Nach dem Merge sollten die roten Dependabot-PRs #4–#6 mit einem Rebase grün werden. Ein formales Approve ist nicht möglich, weil der Bot-Account der PR-Autor ist.

Generated by Claude Code

lukisch · 2026-10-11T07:47:39Z

Zweitmodell-Review: Diff geprüft (nur tests/test_metadata.py). Regex statt fester Action-Majors ist korrekt und entkoppelt den Test von den offenen Dependabot-PRs #4–#6. CI vollständig grün. Freigabe aus Reviewsicht (formales Approve nicht möglich: eigener PR).

Generated by Claude Code

lukisch · 2026-10-11T08:48:37Z

Zweitprüfung (Cloud-Bot): Reiner Testfix (versionsunabhängige Action-Pins per Regex). Alle 10 Checks grün (lint, test 3.11/3.12, Plattform-Smokes, CodeQL). Kein Befund, Freigabe empfohlen.

Generated by Claude Code

lukisch

Zweitmodell-Review (Freigabe technisch nicht möglich, eigener PR): Test-only-Änderung, Action-Pins werden per Regex @v\d+ geprüft. Das entkoppelt die Metadatentests von den Dependabot-Bumps #4–#6. CI auf Head grün (test 3.11/3.12, lint, CodeQL, Platform-Smokes). Aus meiner Sicht mergebar.

Generated by Claude Code

lukisch

Zweitmodell-Review: Freigabe empfohlen. Regex-Pins für checkout/setup-python/github-script korrekt, import re ergänzt, alle 10 Checks grün. (Formale Approval nicht möglich: eigener PR.) Nach Merge Dependabot-PRs #4–#6 per Branch-Update neu laufen lassen.

Generated by Claude Code

lukisch

Zweitmodell-Review: Keine Befunde. Regex-Prüfung @v\d+ entkoppelt die Tests von den Dependabot-Bumps (#4–#6). Freigabefähig; zuerst mergen, dann die Dependabot-PRs.

Generated by Claude Code

lukisch · 2026-10-11T14:49:00Z

Zweitprüfung (Cloud-Bot): CI grün (10/10). Diff minimal und korrekt (assertRegex statt fester Action-Pins).

Diagnose zu den Dependabot-PRs #4–#6: Deren test (3.11/3.12) und linux/macos-platform-smoke schlagen fehl, weil test_metadata die Action-Versionen hart prüft. Dieser PR behebt die Ursache; nach dem Merge ·@·d·ependabot r·ebase auf #4–#6.

Freigabe-Empfehlung: ✅ merge-fähig.

Generated by Claude Code

lukisch · 2026-10-11T18:48:17Z

Zweitreview (Cloud-Bot): Die Regex-Prüfung der Action-Pins statt fester Versionen ist sinnvoll und entkoppelt die Tests von den Dependabot-PRs #4–#6. Alle 10 Checks sind grün. Aus Sicht des Zweitreviews merge-fähig. Eine formale Approval ist nicht möglich, weil der Bot dasselbe Konto wie der Autor nutzt. Nach dem Merge die Dependabot-PRs rebasen.

Generated by Claude Code

lukisch

Zweitmodell-Review: Versionsunabhängige Regex-Prüfung der Action-Pins ist korrekt und macht die Tests gegen die offenen Dependabot-PRs (#4–#6) robust. CI grün. Freigabe empfohlen; danach können die Dependabot-PRs nachziehen.

Generated by Claude Code

lukisch

Zweitmodell-Review (LGTM, mergebereit): Die Regex-Pins (@v\d+) sind korrekt, der Diff beschränkt sich auf tests/test_metadata.py, CI vollständig grün.

Hinweis: Die Dependabot-PRs #4/#5/#6 (checkout 7, github-script 9, setup-python 7) scheitern aktuell genau an den starren Version-Asserts. Nach Merge von #7 dort ·@·d·ependabot r·ebase auslösen.

Generated by Claude Code

lukisch · 2026-10-11T23:48:07Z

Zweitmodell-Review (Cloud-Bot) – Diff geprüft (tests/test_metadata.py), CI vollständig grün. Versionsunabhängige Regex für checkout/setup-python/github-script ist korrekt, import re ergänzt. Freigabefähig; nach Merge sollten die Dependabot-PRs #4–#6 grün durchlaufen (ggf. ·@·d·ependabot r·ebase).

Generated by Claude Code

lukisch

Zweitmodell-Review (Cloud-Bot): Diff geprüft, CI grün. Die Regex-Pins (@v\d+) machen test_metadata unabhängig von den offenen Dependabot-Bumps (#4–#6) – korrekt und minimal. Freigabe empfohlen; danach können #4/#5/#6 gemerged werden.

Generated by Claude Code

lukisch · 2026-10-12T02:48:22Z

Bot-Review (Zweitmodell): Die Regex-Prüfung (@v\d+) entkoppelt test_metadata korrekt von den Dependabot-Bumps #4/#5/#6 – genau der Grund, warum diese sonst rot würden. CI auf Head ist komplett grün (lint, test 3.11/3.12, Plattform-Smokes, CodeQL). Empfehlung: zuerst #7 mergen, dann #4–#6 rebasen/mergen. Freigabe aus Sicht der Prüfung.

Generated by Claude Code

lukisch · 2026-10-12T03:48:12Z

Triage (Cloud-Bot): #7 hat grünes CI (lint, test 3.11/3.12, Linux/macOS-Smoke, CodeQL). Die Dependabot-PRs #4/#5/#6 (setup-python 7, github-script 9, checkout 7) sind rot auf test/Smoke. Das passt zu den Action-Version-Pins in test_metadata, die dieser PR versionsunabhängig macht (aus den Logs nicht abschließend belegt, die Job-Logs zeigen nur den Cleanup-Teil). Empfehlung: #7 zuerst mergen, dann ·@·d·ependabot r·ebase auf #4–#6 und CI neu bewerten.

Generated by Claude Code

lukisch

Zweitmodell-Review (Freigabe-Empfehlung, formales Approve ist beim eigenen PR nicht möglich): Die Regex-Prüfung @v\d+ macht die Action-Pin-Tests unabhängig von Dependabot-Bumps. #4, #5 und #6 würden sonst test_metadata brechen. Die Änderung ist minimal und korrekt, die gesamte CI ist grün. Empfehlung: #7 zuerst mergen, dann die Dependabot-PRs #4, #5 und #6 aktualisieren und mergen.

Generated by Claude Code

lukisch · 2026-10-12T06:49:30Z

Zweitmodell-Review (Freigabe empfohlen): Die Regex-Prüfung (@v\d+) behebt die Ursache der roten Dependabot-PRs #4/#5/#6, denn test_metadata pinnte checkout@v4, setup-python@v5 und github-script@v7. CI auf diesem Head ist komplett grün, der Diff ist minimal und rein testseitig. Nach dem Merge die Dependabot-PRs per ·@·d·ependabot r·ebase neu bauen lassen.

Generated by Claude Code

lukisch commented Oct 11, 2026

View reviewed changes

lukisch commented Oct 12, 2026

View reviewed changes

Conversation

lukisch commented Oct 11, 2026

Uh oh!

github-actions Bot commented Oct 11, 2026

Uh oh!

lukisch commented Oct 11, 2026

Uh oh!

lukisch commented Oct 11, 2026

Uh oh!

lukisch commented Oct 11, 2026

Uh oh!

lukisch left a comment

Choose a reason for hiding this comment

Uh oh!

lukisch left a comment

Choose a reason for hiding this comment

Uh oh!

lukisch left a comment

Choose a reason for hiding this comment

Uh oh!

lukisch commented Oct 11, 2026

Uh oh!

lukisch commented Oct 11, 2026

Uh oh!

lukisch left a comment

Choose a reason for hiding this comment

Uh oh!

lukisch left a comment

Choose a reason for hiding this comment

Uh oh!

lukisch commented Oct 11, 2026

Uh oh!

lukisch left a comment

Choose a reason for hiding this comment

Uh oh!

lukisch commented Oct 12, 2026

Uh oh!

lukisch commented Oct 12, 2026

Uh oh!

lukisch left a comment

Choose a reason for hiding this comment

Uh oh!

lukisch commented Oct 12, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants