Skip to content

Add zizmor to the 2.x branch#52

Merged
Seldaek merged 2 commits into
2.xfrom
add-zizmor-2x
May 29, 2026
Merged

Add zizmor to the 2.x branch#52
Seldaek merged 2 commits into
2.xfrom
add-zizmor-2x

Conversation

@Seldaek

@Seldaek Seldaek commented May 29, 2026

Copy link
Copy Markdown
Member

Brings the 2.x maintenance branch up to the same GitHub Actions security bar as main.

Changes

  • Add .github/workflows/zizmor.yml — zizmor (pedantic persona) security analysis, mirroring main's workflow but with the push trigger scoped to 2.x.
  • Harden the CI, lint and PHPStan workflows so they pass zizmor's pedantic audit:
    • Pin all actions to release SHAs (matching the pins already used on main):
      • actions/checkoutde0fac2 (v6.0.2)
      • shivammathur/setup-php7c071df (2.37.1)
      • ramsey/composer-install65e4f84 (4.0.0)
    • Add persist-credentials: false to read-only checkouts.
    • Add a workflow-level concurrency block.
  • Remove .github/dependabot.yml — GitHub only reads dependabot.yml from the default branch, so the copy on 2.x was dead config (and the sole source of zizmor's dependabot-cooldown finding).

zizmor --persona pedantic .github reports "No findings to report." locally.

🤖 Generated with Claude Code

Seldaek and others added 2 commits May 29, 2026 14:50
Adds a GitHub Actions security analysis (zizmor, pedantic persona) workflow
scoped to the 2.x branch, mirroring the one on main. Also removes the
.github/dependabot.yml that is ignored by GitHub on non-default branches.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Pin actions/checkout, shivammathur/setup-php and ramsey/composer-install to
release SHAs, add persist-credentials: false to checkouts, and add a
workflow-level concurrency block to the CI, lint and PHPStan workflows.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Seldaek Seldaek merged commit 99ec180 into 2.x May 29, 2026
12 of 27 checks passed
@Seldaek Seldaek deleted the add-zizmor-2x branch May 29, 2026 12:54
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