Skip to content

ci: limit GITHUB_TOKEN permissions for lint and check-dist - #332

Merged
andreiborza merged 1 commit into
masterfrom
ab/add-workflow-permissions
Aug 28, 2026
Merged

ci: limit GITHUB_TOKEN permissions for lint and check-dist#332
andreiborza merged 1 commit into
masterfrom
ab/add-workflow-permissions

Conversation

@andreiborza

Copy link
Copy Markdown
Member

What

Adds permissions: contents: read to the lint job in build.yml and the check-dist job in verify-dist.yml. Both only check out the repo and run local commands.

Why

Closes the two open CodeQL alerts for actions/missing-workflow-permissions. These were the last two jobs without an explicit permissions block.

@andreiborza
andreiborza requested review from JPeer264 and msonnb August 28, 2026 12:41
Co-Authored-By: Claude <noreply@anthropic.com>
@andreiborza
andreiborza force-pushed the ab/add-workflow-permissions branch from cd8e6a9 to 00552fd Compare August 28, 2026 12:43
@andreiborza
andreiborza merged commit 2d535e1 into master Aug 28, 2026
37 checks passed
Comment thread action.yml
INPUT_DISABLE_TELEMETRY: ${{ inputs.disable_telemetry }}
INPUT_DISABLE_SAFE_DIRECTORY: ${{ inputs.disable_safe_directory }}
uses: docker://ghcr.io/getsentry/action-release-image:ab-bump-sentry-node-v10
uses: docker://ghcr.io/getsentry/action-release-image:ab-add-workflow-permissions

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

image

@andreiborza
andreiborza deleted the ab/add-workflow-permissions branch August 28, 2026 12:51
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.

3 participants