Skip to content

ci: declare workflow-level contents: read on ci and lint#3704

Merged
chaokunyang merged 1 commit into
apache:mainfrom
arpitjain099:chore/declare-workflow-perms
May 25, 2026
Merged

ci: declare workflow-level contents: read on ci and lint#3704
chaokunyang merged 1 commit into
apache:mainfrom
arpitjain099:chore/declare-workflow-perms

Conversation

@arpitjain099
Copy link
Copy Markdown
Contributor

Both workflows run pure CI / lint checks; no GitHub API writes from the workflows. Workflow-level contents: read is sufficient as the cap for the default GITHUB_TOKEN.

Same post-CVE-2025-30066 (tj-actions/changed-files) hardening pattern. YAML validated locally.

Both workflows run pure CI / lint checks; no GitHub API writes from the workflows. contents: read at workflow level is sufficient.

Post-CVE-2025-30066 hardening shape. yaml.safe_load validated.

Signed-off-by: Arpit Jain <arpitjain099@gmail.com>
Copy link
Copy Markdown
Collaborator

@chaokunyang chaokunyang left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@chaokunyang chaokunyang merged commit af4073e into apache:main May 25, 2026
53 checks passed
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