fix: grant write permissions for review queue sync#2254
Conversation
Up to standards ✅🟢 Issues
|
There was a problem hiding this comment.
Pull request overview
Updates the GitHub Actions workflow token permissions for the Review Queue Label Sync workflow to allow it to write to pull requests (needed for label/PR state updates during the sync job), addressing the 403 Resource not accessible by integration failure during label sync.
Changes:
- Change
pull-requestspermission fromreadtowritein.github/workflows/review-sync.yml.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughThe ChangesWorkflow Pull Request Permissions
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 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 |
exploreriii
left a comment
There was a problem hiding this comment.
Could you rebase with latest changes please, your main is out dated
https://github.com/darshit2308/hiero-sdk-python
this PR should not contain 14 commits from the prior PR
c996045 to
f9eb2cc
Compare
f9eb2cc to
c668d78
Compare
Signed-off-by: darshit2308 <darshit2308@gmail.com>
c668d78 to
b0c539d
Compare
Signed-off-by: darshit2308 <darshit2308@gmail.com>
Signed-off-by: darshit2308 <darshit2308@gmail.com>
Signed-off-by: darshit2308 <darshit2308@gmail.com>
Signed-off-by: darshit2308 <darshit2308@gmail.com>
Description:
Adds
pull-requests: writepermissions toreview-sync.ymlto resolve the403 Resource not accessible by integrationerror encountered during the label sync phase.Notes for reviewer:
Opening a follow-up PR specifically to fix the missing GitHub Actions permissions we discussed.
Checklist