Skip to content

[TASK] Migrate backport to shared reusable workflow#188

Merged
linawolf merged 1 commit into
TYPO3-Documentation:mainfrom
CybotTM:task/migrate-backport-shared-workflow
Jul 7, 2026
Merged

[TASK] Migrate backport to shared reusable workflow#188
linawolf merged 1 commit into
TYPO3-Documentation:mainfrom
CybotTM:task/migrate-backport-shared-workflow

Conversation

@CybotTM

@CybotTM CybotTM commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Summary

blog_example was the last TYPO3-Documentation repo still referencing the broken m-kuhn/backport@30b6e83 action (missing dist/index.js — the original bug that started the shared-workflows initiative). This replaces it with the org shared reusable-backport.yml, matching the 9 already-migrated repos.

Change

-    steps:
-      - name: Backport
-        uses: m-kuhn/backport@30b6e83...   # BROKEN — missing dist/index.js
-        with:
-          github_token: ${{ secrets.GITHUB_TOKEN }}
+    uses: TYPO3-Documentation/.github/.github/workflows/reusable-backport.yml@main
+    with:
+      label_pattern: "^backport (.+)$"
+    secrets:
+      APP_ID: ${{ secrets.APP_ID }}
+      APP_PRIVATE_KEY: ${{ secrets.APP_PRIVATE_KEY }}

⚠️ One thing a maintainer must confirm

The shared workflow generates a GitHub App token from APP_ID / APP_PRIVATE_KEY (so backport PRs trigger their own CI) — it does not fall back to GITHUB_TOKEN. The 9 sibling repos rely on these secrets being available org-wide, but I could not verify that blog_example has access (secret listing is admin-gated). Please confirm blog_example is in the scope of the org APP_ID / APP_PRIVATE_KEY secrets before relying on the first backport. This is not a regression risk: the current action is already broken, so the migration is neutral-or-better either way.

Reference style

Uses @main, matching the open migration PRs (TYPO3-Documentation/TYPO3CMS-Tutorial-Editors#258, TYPO3-Documentation/render-guides#1196) and the recommendation in ADR-003 (TYPO3-Documentation/.github#6). If maintainers decide on SHA-pinning there, this can be pinned to match.

Related

Test plan

  • actionlint clean
  • Maintainer confirms APP_ID/APP_PRIVATE_KEY reach blog_example
  • First labeled+merged PR produces a backport PR (end-to-end)

blog_example was the last TYPO3-Documentation repo still using the broken
m-kuhn/backport@30b6e83 action (missing dist/index.js). Replaces it with the
org shared reusable-backport.yml (korthout/backport-action under the hood),
matching the 9 already-migrated repos. Tracked in TYPO3-Documentation/.github#5.

Signed-off-by: Sebastian Mendel <github@sebastianmendel.de>
@linawolf
linawolf merged commit 78aae23 into TYPO3-Documentation:main Jul 7, 2026
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