Skip to content

feat: secure CI with Zizmor - #36

Merged
Grotax merged 4 commits into
mainfrom
SMillerDev-patch-1
Apr 26, 2026
Merged

Grotax merged 4 commits into
mainfrom
SMillerDev-patch-1

Conversation

@SMillerDev

Copy link
Copy Markdown

Just to make sure we're not doing anything weird here.

@SMillerDev
SMillerDev requested a review from alexdebril as a code owner April 9, 2026 09:39
@Grotax
Grotax requested review from Copilot and removed request for alexdebril April 25, 2026 15:04

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds CI security analysis via Zizmor and tightens GitHub Actions workflow permissions by defaulting to no token permissions and granting least-privilege per job.

Changes:

  • Add a new zizmor GitHub Actions workflow for workflow security analysis.
  • Harden existing workflows by pinning actions to SHAs and setting explicit job-level permissions.
  • Add Dependabot cooldown settings to reduce update frequency.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
.github/workflows/zizmor.yml Adds a Zizmor workflow to analyze GitHub Actions security.
.github/workflows/unit-tests.yml Pins actions by SHA and applies least-privilege permissions for unit tests.
.github/workflows/phpstan-baseline.yml Pins actions by SHA and applies least-privilege permissions for PHPStan baseline checks.
.github/workflows/build-docs.yml Moves permissions to job-level, pins actions by SHA, and keeps docs deployment working.
.github/dependabot.yml Introduces a cooldown period for Dependabot updates.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/zizmor.yml
Comment thread .github/workflows/zizmor.yml
@Grotax
Grotax force-pushed the SMillerDev-patch-1 branch from cf345d5 to f80968c Compare April 26, 2026 05:22
@Grotax
Grotax merged commit 74e2a7e into main Apr 26, 2026
5 checks passed
@Grotax
Grotax deleted the SMillerDev-patch-1 branch April 26, 2026 05:22
@Grotax Grotax added the enhancement New feature or request label Apr 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants