Skip to content

Update large-pr-labeler.yml with permissions - #259

Merged
sparshrestha merged 2 commits into
developfrom
sparshrestha-patch-4
May 27, 2026
Merged

sparshrestha merged 2 commits into
developfrom
sparshrestha-patch-4

Conversation

@sparshrestha

Copy link
Copy Markdown
Collaborator

This pull request updates the workflow configuration in .github/workflows/large-pr-labeler.yml to add explicit permissions required for the workflow to function properly.

Workflow permissions update:

  • Added issues: write and pull-requests: read permissions to ensure the workflow can label pull requests and interact with issues as needed.

Add permissions for issues and pull requests in workflow
@coderabbitai

coderabbitai Bot commented May 27, 2026 •

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: afbeb9b3-92dc-4340-8aa9-e7ec3845ffc9

📥 Commits

Reviewing files that changed from the base of the PR and between 2494928 and 5ca1c6a.

📒 Files selected for processing (1)
  • .github/workflows/large-pr-labeler.yml
🚧 Files skipped from review as they are similar to previous changes (1)
  • .github/workflows/large-pr-labeler.yml

Walkthrough

This PR adds a top-level permissions block to .github/workflows/large-pr-labeler.yml, explicitly granting issues: write and pull-requests: write for the workflow's GitHub token.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding explicit permissions to the large-pr-labeler.yml workflow configuration.
Description check ✅ Passed The description is directly related to the changeset, explaining the workflow permissions update with specific details about the permissions added.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an explicit top-level permissions block to the large-pr-labeler.yml workflow so it can manage the large-pr label on pull requests with least-privilege scopes rather than relying on default GITHUB_TOKEN permissions.

Changes:

  • Declares issues: write to allow creating the repository label and adding/removing it via the issues API.
  • Declares pull-requests: read for reading PR file data.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/large-pr-labeler.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.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.

3 participants