Skip to content

test(ci): accept any major version of actions/github-script - #8

Open
lukisch wants to merge 1 commit into
mainfrom
claude/inspiring-feynman-8waytu
Open

lukisch wants to merge 1 commit into
mainfrom
claude/inspiring-feynman-8waytu

Conversation

@lukisch

@lukisch lukisch commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

Der Test test_lifecycle_workflows_auto_assign_and_label_sync_parity pinnte exakt actions/github-script@v7. Dependabot-PR #6 (v7 → v9) schlägt daran fehl. Der Test akzeptiert jetzt jede Major-Version.

Hinweis: Die analyze-Fehler (CodeQL) in #5/#6/#7 sind davon unabhängig und nicht Teil dieses Fixes. Nach Merge die Dependabot-PRs rebasen (·@·d·ependabot r·ebase).


Generated by Claude Code

…cle parity test

Dependabot bumps (v7 -> v9) failed the exact-pin assertion.

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

github-actions Bot commented Nov 2, 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 Nov 2, 2026

Copy link
Copy Markdown
Contributor Author

CI-Status: Der Check analyze ist rot, aber nicht wegen dieses PRs. Er ist auch auf main rot und scheitert ebenso an #5/#6/#7. Ursache laut Job-Log: „CodeQL analyses from advanced configurations cannot be processed when the default setup is enabled“. Im Repo sind CodeQL Default Setup und der eigene Workflow gleichzeitig aktiv.

Behebung (Repo-Einstellung, nicht Teil dieses PRs): unter Settings → Code security entweder das Default Setup deaktivieren oder den eigenen CodeQL-Workflow entfernen. Die Python-Tests und Analyze (python/actions) sind grün.


Generated by Claude Code

lukisch commented Nov 2, 2026

Copy link
Copy Markdown
Contributor Author

Zweitmodell-Review: Diff korrekt (ein Pin → Regex, import re ergänzt), Python 3.12/3.13 grün. Der rote Check analyze ist ein CodeQL-configuration error (Job-Status 'configuration error'), unabhängig von diesem PR – vermutlich Konflikt zwischen CodeQL-Default-Setup und dem Workflow-Upload in den Repo-Einstellungen; nicht per Code behebbar. Merge empfohlen, danach Dependabot #5–#7 rebasen.


Generated by Claude Code

lukisch commented Nov 2, 2026

Copy link
Copy Markdown
Contributor Author

Review (Zweitmodell): Änderung ist korrekt und minimal (Regex statt fester Version github-script@v7, import re ergänzt); Python 3.12/3.13-Tests, CodeQL-Analyze (python/actions) grün. Der rote Job analyze ist nicht PR-bedingt: "CodeQL analyses from advanced configurations cannot be processed when the default setup is enabled" – im Repo ist CodeQL Default Setup aktiv und kollidiert mit dem Workflow. Fix: entweder Default Setup in den Code-Security-Einstellungen deaktivieren oder den Advanced-Workflow entfernen. Code-seitig freigabefähig.


Generated by Claude Code

@lukisch lukisch left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Zweitprüfung: freigabefähig. Diff minimal (Regex statt exaktem github-script@v7-Pin), Python 3.12/3.13 grün. Rote analyze-Prüfung ist ein CodeQL-Konfigurationsfehler ("advanced configuration cannot be processed when default setup is enabled"), unabhängig von diesem PR. Empfehlung: eigenen CodeQL-Workflow entfernen oder Default-Setup deaktivieren.


Generated by Claude Code

lukisch commented Nov 2, 2026

Copy link
Copy Markdown
Contributor Author

Review (Zweitmodell): Die Regex actions/github-script@v\d+ löst den Versions-Pin im Test sauber. Die Python-Tests sind grün. Der rote analyze-Check ist CodeQL, nicht der Testfix. Im Log steht "overlay analysis previously failed", das sieht nach einem Infrastrukturproblem aus. Nach dem Merge reicht ein Re-Run. Danach die Dependabot-PRs #5, #6 und #7 rebasen.


Generated by Claude Code

@lukisch lukisch left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Zweitprüfung: Teständerung (Regex statt fixer github-script@v7) ist korrekt und passt zu den Dependabot-Bumps #6/#7. Python 3.12/3.13 grün. Einziger roter Check analyze ist ein Repo-Konfigurationsproblem, kein PR-Defekt: "CodeQL analyses from advanced configurations cannot be processed when the default setup is enabled". Fix: im Repo entweder CodeQL-Default-Setup deaktivieren oder den Advanced-Workflow entfernen. Code-seitig nichts zu ändern.


Generated by Claude Code

lukisch commented Nov 3, 2026

Copy link
Copy Markdown
Contributor Author

Zweitprüfung: Der Test-Fix ist korrekt (re.search(r"actions/github-script@v\d+", ...) statt exaktem @v7). Python 3.12 und 3.13 sind grün.

Der einzige rote Check analyze (CodeQL) bricht laut Log mit „CodeQL job status was configuration error" ab. Das ist ein Repo-Konfigurationsproblem und kein Codefehler. Wahrscheinlich kollidiert der Advanced-Workflow mit dem aktivierten „Default setup". Lösung: unter Settings → Code security entweder das Default setup deaktivieren oder den Workflow-Job entfernen. Das kann ich per Code nicht beheben. Der PR selbst ist freigabefähig. Nach dem Merge sollten die Dependabot-PRs #5/#6/#7 rebased werden.


Generated by Claude Code

lukisch commented Nov 3, 2026

Copy link
Copy Markdown
Contributor Author

Review (Zweitmodell, nur Kommentar – keine Freigabe/Merge): Der Test-Fix (Regex actions/github-script@v\d+ statt fixer v7) ist korrekt und nötig, da Dependabot (#6) auf v9 hebt; Python-3.12/3.13-Tests sind grün. Der einzige rote Check analyze (CodeQL, 'configuration error') ist kein PR-Defekt: der CodeQL-Workflow schlägt auch auf main fehl. Vermutlich kollidiert die Advanced-Workflow-Datei codeql.yml mit aktiviertem CodeQL-'Default setup' in den Repo-Einstellungen – bitte eines von beiden deaktivieren.


Generated by Claude Code

lukisch commented Nov 3, 2026

Copy link
Copy Markdown
Contributor Author

Zweitmodell-Review: Die Änderung am Test (re.search(r"actions/github-script@v\d+", ...)) ist korrekt und minimal. Aus meiner Sicht ist sie freigabefähig.

Der rote analyze-Check ist kein Code-Problem. Code Scanning meldet: "CodeQL analyses from advanced configurations cannot be processed when the default setup is enabled". Es laufen also der Default-Setup und der Workflow codeql parallel. Das Repo-Owner-Entscheid wäre, entweder das Default-Setup abzuschalten oder den Workflow zu entfernen. Der Workflow nutzt außerdem codeql-action@v3, das im Dezember 2026 ausläuft (v4 nötig). Das betrifft auch #5/#6/#7.


Generated by Claude Code

lukisch commented Nov 3, 2026

Copy link
Copy Markdown
Contributor Author

CI-Diagnose (Cloud-Bot): Alle Tests und CodeQL-Jobs sind grün. Nur der Workflow-Job analyze scheitert: "CodeQL analyses from advanced configurations cannot be processed when the default setup is enabled". Das ist ein Repo-Konfigurationskonflikt (CodeQL Default Setup und eigener Workflow), kein Codefehler dieses PRs. Fix: entweder Default Setup in den Repo-Einstellungen (Code security) deaktivieren oder den eigenen CodeQL-Workflow entfernen. Zusätzlich nutzt der Workflow codeql-action@v3 (Deprecation Dez 2026), Upgrade auf v4 empfohlen.


Generated by Claude Code

@lukisch lukisch left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Second-model review: LGTM. The test now accepts any actions/github-script@vN (regex), so Dependabot bumps #6 no longer break the contract. Python 3.12/3.13 tests and CodeQL pass. The red analyze check is a CodeQL "configuration error" in the job log, not a test or code failure, so it is unrelated to this diff (likely a conflict with default setup); worth checking the repo's code-scanning settings separately.


Generated by Claude Code

lukisch commented Nov 3, 2026

Copy link
Copy Markdown
Contributor Author

Zweitprüfung (Cloud-Bot): Die Änderung ist minimal und korrekt. re.search(r"actions/github-script@v\d+", ...) akzeptiert alle Major-Versionen und deckt damit die offenen Dependabot-PRs #6 und #7 mit ab. Der analyze-Check ist im letzten Lauf fehlgeschlagen. Das gehört zum Workflow-Lauf des alten Heads und nicht zu dieser Testzeile; bitte nach einem Rebase neu prüfen.


Generated by Claude Code

@lukisch lukisch left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Zweitprüfung (freigabefähig): Test-only, Regex actions/github-script@v\d+ korrekt; Python 3.12/3.13 grün. Roter Check analyze ist unabhängig vom Diff: „CodeQL analyses from advanced configurations cannot be processed when the default setup is enabled“ – Owner-Entscheidung: entweder CodeQL-Default-Setup abschalten oder den Advanced-Workflow entfernen.


Generated by Claude Code

lukisch commented Nov 3, 2026

Copy link
Copy Markdown
Contributor Author

Triage (Cloud-Bot): Der rote Check analyze (CodeQL) endet mit CodeQL job status was configuration error – das ist ein Repo-Konfigurationsproblem (vermutlich Konflikt zwischen CodeQL „Default setup“ und dem Workflow-basierten Setup), nicht durch diesen PR verursacht (er ändert nur den Test für actions/github-script). Empfehlung: in den Repo-Settings (Code security → Code scanning) entweder Default setup oder den Workflow deaktivieren. Der PR selbst ist inhaltlich unkritisch.


Generated by Claude Code

@lukisch lukisch left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Review (Zweitmodell): Regex-Lockerung auf beliebige github-script@vN ist korrekt und entkoppelt den Test von Dependabot-Bumps. Python 3.12/3.13 grün. Der rote analyze-Check ist ein CodeQL-"configuration error" (vermutlich Konflikt Default-Setup vs. Workflow) und nicht durch diesen Diff verursacht – separat im Repo-Security-Setup prüfen.


Generated by Claude Code

This branch has not been deployed

No deployments
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