Skip to content

fix(ci): Resolve zizmor security findings in GitHub Actions workflows - #527

Merged
lahirumaramba merged 2 commits into
mainfrom
lm-zizmor-fixes
Jul 8, 2026
Merged

fix(ci): Resolve zizmor security findings in GitHub Actions workflows#527
lahirumaramba merged 2 commits into
mainfrom
lm-zizmor-fixes

Conversation

@lahirumaramba

@lahirumaramba lahirumaramba commented Jul 8, 2026

Copy link
Copy Markdown
Member

This change resolves security findings diagnosed by zizmor (artipacked and excessive-permissions) across all GitHub Actions workflow YAML files (ci.yml, nightly.yml, and release.yml). Specifically, least-privilege permissions blocks have been added to the workflow and job definitions, and persist-credentials: false has been configured on git checkouts where git credentials are not required by subsequent steps.

This change resolves security findings diagnosed by zizmor (artipacked and excessive-permissions) across all GitHub Actions workflow YAML files (ci.yml, nightly.yml, and release.yml). Specifically, least-privilege permissions blocks have been added to the workflow and job definitions, and persist-credentials: false has been configured on git checkouts where git credentials are not required by subsequent steps.

Testing: Verified all workflows locally using zizmor .github/workflows with 0 findings reported.

Context Sources Used:
- id: firebase-admin-dotnet
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@lahirumaramba
lahirumaramba merged commit b753a8e into main Jul 8, 2026
16 checks passed
@lahirumaramba
lahirumaramba deleted the lm-zizmor-fixes branch July 8, 2026 14:26
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