Skip to content

ci: bump GitHub Actions - #2706

Merged
dunglas merged 5 commits into
mainfrom
ci/bump-actions
Oct 10, 2026
Merged

dunglas merged 5 commits into
mainfrom
ci/bump-actions

Conversation

@dunglas

@dunglas dunglas commented Oct 10, 2026 •

Copy link
Copy Markdown
Member

Fixes the macOS test job, which has failed on every run since 2026-10-08 (including on main: https://github.com/php/frankenphp/actions/runs/37898109529) with Could not setup PHP 8.5. We were pinned to shivammathur/setup-php 2.37.2; 2.40.0 includes the macOS Homebrew alias resolution fix (shivammathur/setup-php@7d671baa).

Also bumps the other outdated actions to their latest releases:

  • actions/cache 4.3.0 → 6.1.0 (watcher action, aligns with the other usages)
  • actions/download-artifact 8.0.1 → 8.0.2
  • actions/setup-go 6.4.0 → 7.0.0
  • actions/upload-artifact 7.0.1 → 7.0.2
  • docker/bake-action 7.3.0 → 7.4.0
  • docker/setup-buildx-action 4.3.0 → 4.4.1
  • super-linter/super-linter 8.7.0 → 9.0.0 (breaking changes only affect stylelint, jscpd and nbqa, none used here)

Dependabot now also scans the composite actions in .github/actions/*: with directory: / it only covered .github/workflows, which is why setup-php, setup-go and the watcher's actions/cache were never bumped.

super-linter 9 ships zizmor 1.30, which adds two audits:

  • self-repository (use uses: $/... for local actions): disabled, because actionlint does not support this syntax yet (Support the new $/ self-repository uses: syntax rhysd/actionlint#711).
  • unsound-ternary: the push && '' || format(...) expressions in docker.yaml and static.yaml always evaluated to the cache flags because '' is falsy, so published images (dunglas/frankenphp and dunglas/frankenphp-dev) and release assets reused cached layers. Builds that push no longer read the cache; every build still writes it, so PR builds keep reusing layers from main.

2.37.2 can no longer install PHP on macOS runners.
actionlint, also run by super-linter, does not support the `$/` syntax yet.
@dunglas
dunglas force-pushed the ci/bump-actions branch 2 times, most recently from 60bbcd0 to 1a24e88 Compare October 10, 2026 08:52
`push && '' || x` always evaluates to `x` because `''` is falsy, so
published images reused cached layers. Skip cache-from when pushing and
always write the cache.
@dunglas
dunglas merged commit 2e904ef into main Oct 10, 2026
41 checks passed
@dunglas
dunglas deleted the ci/bump-actions branch October 10, 2026 15:12
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