Skip to content

ci: declare least-privilege permissions on the 4 remaining workflows#354

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

ci: declare least-privilege permissions on the 4 remaining workflows#354
arpitjain099 wants to merge 1 commit into
cncf:mainfrom
arpitjain099:ci/add-permissions

Conversation

@arpitjain099
Copy link
Copy Markdown

Pins the default GITHUB_TOKEN on the four workflows still inheriting org defaults:

  • format-check.yml, link-check.yml, spell-check.ymlcontents: read. PR-time read-only checks (npm run check:format/markdown/links/spelling).
  • trigger-contribute-site-netlify.ymlpermissions: {}. The job only POSTs to NETLIFY_CONTRIBUTE_SITE_BUILD_HOOK; no checkout, no GitHub API calls.

YAML validated locally.

@netlify
Copy link
Copy Markdown

netlify Bot commented May 13, 2026

Deploy Preview for cncf-techdocs ready!

Name Link
🔨 Latest commit 82c0903
🔍 Latest deploy log https://app.netlify.com/projects/cncf-techdocs/deploys/6a048b3441fd1a0008dec54f
😎 Deploy Preview https://deploy-preview-354--cncf-techdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

- format-check, link-check, spell-check: contents: read (PR-time
  read-only checks).
- trigger-contribute-site-netlify: permissions: {} (only POSTs to the
  Netlify build webhook).

Signed-off-by: arpitjain099 <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.

1 participant