Skip to content

ci: declare token permissions for jobs that write - #4803

Merged
aly-obol merged 1 commit into
mainfrom
ci/explicit-token-permissions
Oct 22, 2026
Merged

aly-obol merged 1 commit into
mainfrom
ci/explicit-token-permissions

Conversation

@aly-obol

@aly-obol aly-obol commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Refs ObolNetwork/obol-infrastructure#3294

The org default GITHUB_TOKEN is read+write, and these jobs rely on it to write (push images to ghcr.io, open or comment on PRs, deploy Pages, upload code scanning results, write check runs). This adds job-level permissions: with exactly what each job needs, so the org default can be switched to read-only without breaking them.

No behaviour change while the org default is still read+write.

category: misc
ticket: none

These jobs rely on the org default GITHUB_TOKEN (read+write). Declaring what they need lets the org default become read-only.

Refs ObolNetwork/obol-infrastructure#3294
@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecov Bot commented Oct 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.61%. Comparing base (bd4e892) to head (3c3a25a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4803   +/-   ##
=======================================
  Coverage   67.61%   67.61%           
=======================================
  Files         254      254           
  Lines       33609    33609           
=======================================
+ Hits        22725    22726    +1     
+ Misses      10883    10882    -1     
  Partials        1        1           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@aly-obol
aly-obol merged commit 4e4d996 into main Oct 22, 2026
12 of 14 checks passed
@aly-obol
aly-obol deleted the ci/explicit-token-permissions branch October 22, 2026 21:22
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