Skip to content

ci: grant actions:write for the DEC-074 artifact prune - #21

Merged
omar-nahhas merged 1 commit into
mainfrom
chore/dec-074-actions-write
Aug 3, 2026
Merged

ci: grant actions:write for the DEC-074 artifact prune#21
omar-nahhas merged 1 commit into
mainfrom
chore/dec-074-actions-write

Conversation

@omar-nahhas

Copy link
Copy Markdown
Contributor

One-line permission grant so the devkit's new count-based artifact retention (keep the last 5 runs) actually works in this repo.

This repo calls the reusable plugin-e2e.yml@main, so it picked up the prune step the moment testkit#49 merged. A reusable workflow's job permissions are capped by the caller's grant — without actions: write here the prune logs a warning and deletes nothing, so e2e artifacts keep accumulating.

Non-fatal either way (the prune step is continue-on-error), but until this lands the feature is inert here.

devkit-sync cannot deliver this: it deliberately never writes .github/workflows/. Equivalent to re-running make link-workflows.

The file now matches templates/plugin-e2e.caller.yml byte-for-byte.

The devkit's reusable plugin-e2e now enforces count-based artifact retention
(keep the last 5 runs) by DELETING superseded artifacts via the Actions API.
A reusable workflow's job permissions are CAPPED by the caller's grant, so
without actions:write here the prune degrades to a warning and artifacts
accumulate unbounded.

Mirrors templates/plugin-e2e.caller.yml. devkit-sync cannot deliver this — it
deliberately never writes .github/workflows/.
@omar-nahhas
omar-nahhas merged commit e56f1ee into main Aug 3, 2026
3 checks 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.

1 participant