Skip to content

ci: declare contents: read on ci.yml and codespell.yml#296

Open
arpitjain099 wants to merge 1 commit into
containerd:mainfrom
arpitjain099:ci/add-permissions
Open

ci: declare contents: read on ci.yml and codespell.yml#296
arpitjain099 wants to merge 1 commit into
containerd:mainfrom
arpitjain099:ci/add-permissions

Conversation

@arpitjain099
Copy link
Copy Markdown

Two workflows currently inherit the default GITHUB_TOKEN scope:

  • ci.yml (jobs: checks, test-build, linters, tests) — checkout + Go build/test.
  • codespell.yml (job: codespell) — checkout + codespell-project/actions-codespell scan.

Neither pushes commits, opens issues, or calls write APIs. contents: read at the workflow level is the right minimum. codeql.yml in this repo already follows the same workflow-level pattern; this brings the remaining two CI workflows in line.

Both workflows do checkout + build/test (ci.yml) or checkout + codespell
scan (codespell.yml). Neither pushes commits or calls write APIs.
codeql.yml in this repo already declares workflow-level contents: read;
this brings the remaining two in line.

Signed-off-by: Arpit Jain <arpitjain099@gmail.com>
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