Skip to content

fix: replace inline heredoc with standalone Python script - #44

Merged
oaustegard merged 2 commits into
mainfrom
claude/github-ai-readme-action-beNGA
Mar 3, 2026
Merged

fix: replace inline heredoc with standalone Python script#44
oaustegard merged 2 commits into
mainfrom
claude/github-ai-readme-action-beNGA

Conversation

@oaustegard

Copy link
Copy Markdown
Owner

No description provided.

claude added 2 commits March 3, 2026 01:35
The <<'PYEOF' shell heredoc inside a YAML run block was causing
GitHub Actions to reject the workflow (0s failure, no logs) — the
<< character conflicts with YAML's merge key operator.

Changes:
- Extract Python logic to .github/scripts/generate_readme.py
  (can also be run locally for testing)
- Simplify the matrix to a single generate-readmes job that loops
  over files with a plain shell for-loop
- Workflow YAML is now free of heredocs and easier to read
…ator

The models:read permission scope (added April 2025) appears to be
unknown to GitHub's YAML schema validator, causing immediate 0s
failure before any job runs. Removing it so the workflow passes
validation. The GITHUB_TOKEN can still call GitHub Models if Models
is enabled for the repository; a 403 at runtime (with logs) is far
easier to diagnose than a silent schema rejection.
@oaustegard
oaustegard merged commit 6f8f336 into main Mar 3, 2026
1 check passed
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