Skip to content

ci: first-interaction v3 Eingaben mit Unterstrichen verwenden - #2

Merged
lukisch merged 1 commit into
masterfrom
claude/amazing-fermi-f2io7m
Oct 4, 2026
Merged

lukisch merged 1 commit into
masterfrom
claude/amazing-fermi-f2io7m

Conversation

@lukisch

@lukisch lukisch commented Oct 3, 2026

Copy link
Copy Markdown
Contributor

Der Welcome-Workflow nutzte bereits @v3, aber Eingaben mit Bindestrich (repo-token, issue-message, pr-message). Laut action.yml am Tag v3 heißen sie repo_token, issue_message, pr_message. Angepasst.

Nicht lokal testbar (GitHub-Actions-Workflow); wird beim nächsten Issue/PR ausgelöst.

🤖 Generated with Claude Code

https://claude.ai/code/session_01VnxPUjm2uYvYzB7PEC3ZFL


Generated by Claude Code

lukisch commented Oct 3, 2026

Copy link
Copy Markdown
Contributor Author

Der rote welcome-Check auf diesem PR ist erwartet und kein Fehler der Änderung. Der Workflow läuft auf pull_request_target und führt deshalb die Version vom Base-Branch (master) aus, nicht die aus diesem PR. Das Job-Log zeigt dort noch die alten Eingaben (Unexpected input(s) 'repo-token', 'issue-message', 'pr-message' → Input required and not supplied: issue_message). Die korrigierten Unterstrich-Eingaben aus diesem PR greifen erst nach dem Merge. Ein erneuter Lauf würde auf diesem PR dasselbe Ergebnis liefern, daher keine Wiederholung. Alle anderen Checks sind unabhängig davon.


Generated by Claude Code

lukisch commented Oct 3, 2026

Copy link
Copy Markdown
Contributor Author

Review (Zweitmodell): Der Diff ist korrekt (repo_token/issue_message/pr_message für first-interaction@v3). Das Log des roten welcome-Jobs zeigt noch die alten Bindestrich-Inputs (Unexpected input(s) 'repo-token', …, issue_message fehlt), er läuft also mit dem Workflow aus dem Base-Branch und nicht mit dieser Änderung. Er sollte nach dem Merge grün werden. pytest und Smoke sind grün. Aus meiner Sicht merge-fähig.


Generated by Claude Code

@lukisch
lukisch merged commit 0e1fc30 into master Oct 4, 2026
6 of 7 checks passed
@lukisch
lukisch deleted the claude/amazing-fermi-f2io7m branch October 4, 2026 09:07
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