Skip to content

ci: adopt deny-all top-level workflow permissions#155

Merged
mscolnick merged 1 commit into
mainfrom
chore/deny-all-workflow-permissions
Jul 14, 2026
Merged

ci: adopt deny-all top-level workflow permissions#155
mscolnick merged 1 commit into
mainfrom
chore/deny-all-workflow-permissions

Conversation

@mscolnick

Copy link
Copy Markdown
Contributor

Sets each workflow's top-level permissions: to {} (deny-all) and moves the previously top-level grants into the specific jobs that inherited them. Effective permissions are unchanged for every job — this just makes the grants explicit per job so a newly-added job can't silently inherit broad scopes.

Behavior-preserving mechanical change (validated: all touched workflows still parse; per-job effective scopes identical). Workflows whose jobs inherit the org default are left untouched for a manual per-job pass.

Flagged by github-maintenance Workflow Permissions Audit.

Set each workflow's top-level `permissions:` to `{}` (deny-all) and move the
previously top-level grants into the specific jobs that inherited them.
Effective permissions are unchanged for every job — this makes the grants
explicit per job so a newly-added job can't silently inherit broad scopes.

Flagged by github-maintenance Workflow Permissions Audit.
Copilot AI review requested due to automatic review settings July 14, 2026 20:45

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@mscolnick
mscolnick merged commit 749da92 into main Jul 14, 2026
4 checks passed
@mscolnick
mscolnick deleted the chore/deny-all-workflow-permissions branch July 14, 2026 20:52
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