Skip to content

Screenshots: push with a deploy key past a ruleset - #43

Merged
firsttris merged 1 commit into
mainfrom
screenshots-deploy-key
Nov 8, 2026
Merged

firsttris merged 1 commit into
mainfrom
screenshots-deploy-key

Conversation

@firsttris

Copy link
Copy Markdown
Owner

Why

Same problem as #42: in firsttris/haben the ruleset on main (pull request, status checks) refuses the GITHUB_TOKEN, so Update screenshots on main can no longer push its commit.

What

  • screenshots.yml: optional secret deploy-key, passed to actions/checkout as ssh-key. commit-changes then pushes over SSH with that key. Without the secret nothing changes.
  • The Playwright image (mcr.microsoft.com/playwright:vX-noble) installs git openssh-client (Dockerfile.noble), so SSH works in the job's container.
  • README: secret table and a note that the same deploy key as for Bump version can be used, and that its push runs the branch's CI.

Checks

YAML parses; actionlint runs in Lint. Not run end to end yet: that happens with the first run in haben after this is merged.


Generated by Claude Code

Like bump-version.yml (#42): the optional secret deploy-key checks out
and pushes with a deploy key on the ruleset's bypass list. The
Playwright image brings openssh-client (utils/docker/Dockerfile.noble
in microsoft/playwright).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Y3rdfXpBafq4kFbwiY2UFF
@firsttris
firsttris merged commit e3bd464 into main Nov 8, 2026
7 of 8 checks passed
@firsttris
firsttris deleted the screenshots-deploy-key branch November 8, 2026 10:58
firsttris added a commit to firsttris/haben that referenced this pull request Nov 8, 2026
## Anlass

Wie beim Bump (#397, #398): Das Ruleset
auf `main` lehnt Pushes mit dem `GITHUB_TOKEN` ab, deshalb kann *Update
screenshots* auf `main` seinen Commit nicht mehr pushen.

## Änderung

- `.github/workflows/screenshots.yml`: gepinnt auf `e3bd464`, den
Merge-Commit von firsttris/workflows#43 auf `main`. Übergibt
`BUMP_DEPLOY_KEY` als `deploy-key`. Das ist derselbe Deploy Key wie beim
Bump, er steht schon auf der Bypass-Liste. Den Kommentar am Pin habe ich
von `# v1` auf `# main` geändert, weil der Commit nicht auf dem Tag `v1`
liegt.
- `docs/entwicklung.md`: ein Satz zum Push auf `main` mit dem Deploy
Key.

Im Playwright-Image ist `openssh-client` installiert, deshalb
funktioniert der Deploy Key auch im Container. Getestet ist das erst mit
dem ersten Lauf auf `main` nach dem Merge. Der Commit mit den Bildern
startet dann die CI auf `main`.

---
_Generated by [Claude
Code](https://claude.ai/code/session_01Y3rdfXpBafq4kFbwiY2UFF)_

Co-authored-by: Claude <noreply@anthropic.com>
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