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

Zweitmodell-Review: Freigabe-Empfehlung. Die Regex-Prüfung (@v\d+) macht die Metadaten-Tests unabhängig von Dependabot-Bumps und löst damit die roten Tests der offenen Dependabot-PRs #4/#5/#6. CI vollständig grün (test 3.11/3.12, lint, smokes, CodeQL). Empfehlung: #7 zuerst mergen, dann #4–#6 rebasen/mergen.

Generated by Claude Code

lukisch · 2026-10-12T09:49:11Z

Second-model review: the diff only touches tests/test_metadata.py. It replaces the pinned action versions with @v\d+ regexes, and CI is fully green (test 3.11/3.12, lint, platform smoke, CodeQL). I am commenting instead of approving because the bot account cannot approve a PR authored by the same user. No objections to merging. After the merge, please run ·@·d·ependabot r·ebase on #4, #5 and #6, because they currently fail only because of the pinned versions in the test.

Generated by Claude Code

lukisch

Zweitmodell-Review (Freigabe-Empfehlung): Die Regex-Prüfung (actions/<name>@v\d+) entkoppelt test_metadata von den Dependabot-Bumps (#4, #5, #6). Diff minimal, re ist importiert, alle 10 Checks (lint, test 3.11/3.12, Platform-Smokes, CodeQL) grün. Kein Einwand gegen den Merge. Danach die Dependabot-PRs rebasen lassen.

(Formales Approve nicht möglich: gleicher Account wie PR-Autor.)

Generated by Claude Code

lukisch · 2026-10-12T14:49:39Z

Zweitreview (Cloud-Bot): Alle 10 Checks grün (test 3.11/3.12, lint, Plattform-Smokes, CodeQL). Die Regex-Prüfung @v\d+ ersetzt starre Action-Pins sinnvoll und verhindert Dependabot-Konflikte. Keine Befunde – freigabefähig.

Generated by Claude Code

lukisch

Zweitmodell-Review: Alle 10 Checks grün (lint, test 3.11/3.12, Plattform-Smokes, CodeQL). Die Änderung macht die Action-Pin-Tests versionsunabhängig (Regex @v\d+) und ist klein und korrekt – dependabot-Bumps brechen die Tests damit nicht mehr. Freigabe-Empfehlung: merge.

Generated by Claude Code

lukisch

Zweitprüfung (Freigabe-Empfehlung): Diff minimal (nur tests/test_metadata.py, Regex statt fester Action-Versionen), alle 10 Checks grün. Behebt die Ursache der roten Dependabot-PRs #4/#5/#6. Nach dem Merge dort ·@·d·ependabot r·ebase. (Formale Approval nicht möglich: gleicher Account wie Autor.)

Generated by Claude Code

lukisch · 2026-10-12T17:48:24Z

Zweitmodell-Review (Cloud-Bot)

Die Änderung ist korrekt und CI ist grün. Die Action-Pins werden jetzt versionsunabhängig geprüft. Damit lassen sich die Dependabot-PRs #4, #5 und #6 (checkout 7, setup-python 7, github-script 9) nach dem Merge von #7 ohne Testbruch mergen. Bitte #7 zuerst mergen, danach die Dependabot-PRs rebasen. Die Dependabot-PRs sind derzeit ohne Statuschecks (#6: pending), also auf CI nach dem Rebase achten.

Freigabe-Empfehlung: merge.

Generated by Claude Code

lukisch

Zweitmodell-Review: LGTM. Die Regex-Prüfung (@v\d+) macht test_metadata unabhängig von Dependabot-Bumps und behebt die roten Checks in #4/#5/#6. Nur Testcode betroffen; CI (lint, test 3.11/3.12, Smoke, CodeQL) ist grün. Nach dem Merge die Dependabot-PRs per ·@·d·ependabot r·ebase aktualisieren.

Generated by Claude Code

lukisch

Zweitmodell-Review: Freigabe aus meiner Sicht. Die Regex-Prüfungen (@v\d+) machen test_metadata versionsunabhängig, CI ist vollständig grün. Empfehlung: #7 zuerst mergen, danach die Dependabot-PRs #4, #5 und #6 (checkout 4→7, setup-python 5→7, github-script 7→9). Sie würden sonst an den bisherigen exakten Pins im Test scheitern.

Generated by Claude Code

lukisch

Zweitmodell-Review: freigabereif. Die Tests prüfen Action-Pins jetzt per Regex (@v\d+) statt mit festen Versionen. Das behebt die Kollision mit den Dependabot-Bumps #4–#6, ohne die Prüfung abzuschwächen. Alle 10 Checks sind grün.

Hinweis: #4–#6 nach dem Merge von #7 per Update-Branch nachziehen, damit test_metadata dort grün wird.

Generated by Claude Code

lukisch · 2026-10-12T22:48:26Z

Zweitreview (Cloud-Bot): CI grün, Änderung minimal und korrekt. Die versionsunabhängigen Regexe entsperren die offenen Dependabot-PRs #4–#6 (checkout→7, setup-python→7, github-script→9), deren Tests sonst an den harten @v4/@v5/@v7-Pins scheitern würden. Empfehlung: zuerst #7 mergen, dann die Dependabot-PRs rebasen lassen. Keine Befunde.

Generated by Claude Code

lukisch commented Oct 11, 2026

View reviewed changes

lukisch commented Oct 12, 2026

View reviewed changes

lukisch mentioned this pull request Oct 12, 2026

chore(deps): bump actions/checkout from 4 to 7 #6

Open

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!

lukisch left a comment

Choose a reason for hiding this comment

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!

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 12, 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 12, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!