ci: pin contents: read on mixin, publish, release, ui-ci#5237
ci: pin contents: read on mixin, publish, release, ui-ci#5237arpitjain099 wants to merge 1 commit into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (4)
✅ Files skipped from review due to trivial changes (2)
🚧 Files skipped from review as they are similar to previous changes (2)
📝 WalkthroughWalkthroughFour GitHub Actions workflows now declare top-level ChangesGitHub Actions Permission Hardening
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
All four workflows: contents: read. Image pushes use docker_hub + quay creds; the GitHub release publish step in release.yml uses PROMBOT_GITHUB_TOKEN. The default GITHUB_TOKEN is only used for the checkout. Signed-off-by: arpitjain099 <arpitjain099@gmail.com>
033096f to
c79c481
Compare
Pins the default
GITHUB_TOKENto read-only on the four workflows still inheriting org defaults. All four are read-only from GitHub's perspective:mixin.yml—make -C doc/alertmanager-mixin lint.publish.yml— main-branch build + Docker Hub / Quay image push usingdocker_hub_login/docker_hub_password+quay_io_login/quay_io_password.release.yml— tag build + image push (same creds), plus a GitHub release viaPROMBOT_GITHUB_TOKEN.ui-ci.yml—npm run testfor the mantine-ui workspace.No dependency on the default
GITHUB_TOKENfor writes. YAML validated locally.Summary by CodeRabbit