Skip to content

ci: add explicit permissions to GitHub Actions workflows#6440

Merged
wjones127 merged 1 commit intolance-format:mainfrom
wjones127:workflow-permissions
Apr 8, 2026
Merged

ci: add explicit permissions to GitHub Actions workflows#6440
wjones127 merged 1 commit intolance-format:mainfrom
wjones127:workflow-permissions

Conversation

@wjones127
Copy link
Copy Markdown
Contributor

Most workflows lacked a permissions block, causing GitHub security warnings. Added permissions: contents: read at the top level for all affected workflows.

Special cases:

  • benchmark-comment-trigger: also needs pull-requests: read to call the pulls REST API
  • nightly_run: run job needs actions: write to dispatch file_verification.yml
  • rust: clippy job-level permissions updated to include contents: read alongside checks: write
  • cargo-publish: build job updated to include contents: read alongside id-token: write

Workflows already having correct permissions (claude.yml, claude-code-review.yml, pr-title.yml, stale.yml, rust-benchmark.yml, docs-deploy.yml, codex-fix-ci.yml, codex-backport-pr.yml, file_verification.yml, cargo-publish.yml) were left unchanged or minimally updated.

Most workflows lacked a `permissions` block, causing GitHub security
warnings. Added `permissions: contents: read` at the top level for all
affected workflows.

Special cases:
- `benchmark-comment-trigger`: also needs `pull-requests: read` to call the pulls REST API
- `nightly_run`: `run` job needs `actions: write` to dispatch `file_verification.yml`
- `rust`: `clippy` job-level permissions updated to include `contents: read` alongside `checks: write`
- `cargo-publish`: `build` job updated to include `contents: read` alongside `id-token: write`
@github-actions github-actions Bot added the ci Github Action or Test issues label Apr 8, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@wjones127 wjones127 marked this pull request as ready for review April 8, 2026 17:59
@wjones127 wjones127 merged commit d503b8e into lance-format:main Apr 8, 2026
42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Github Action or Test issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants