Skip to content

Conversation

@Gatsik
Copy link
Contributor

@Gatsik Gatsik commented Dec 20, 2025

There is an action to draft PRs automatically which fails if a PR is not already a draft because of insufficient permissions
See #6460 (comment)

Summary by CodeRabbit

  • Chores
    • Updated GitHub Actions workflow permissions configuration to enable automated processes.

✏️ Tip: You can customize this high-level summary in your review settings.

@Gatsik Gatsik marked this pull request as draft December 20, 2025 11:54
@coderabbitai
Copy link

coderabbitai bot commented Dec 20, 2025

Walkthrough

Adds a contents: write permission declaration to the GitHub Actions workflow file .github/workflows/auto-draft-pr.yaml, enabling the workflow to write to repository contents. No functional changes to workflow steps or environment variables.

Changes

Cohort / File(s) Change Summary
GitHub Actions Workflow Permissions
​.github/workflows/auto-draft-pr.yaml
Added contents: write permission to the workflow permissions block to grant write access to repository contents

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • Minimal configuration change requiring verification that the permission level is appropriate for the workflow's intended operations

Poem

🐰 A permission slip signed with care,
Write to contents, now we're there!
Workflows dancing, actions flow,
Permissions granted, watch us go! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description explains the problem and references the related issue, but lacks detailed testing information and changelog documentation as required by the template. Add 'Testing done on the proposed changes' section, 'Additional context' section, and update the checklist to confirm annotations, changelog entry, and reviewer assignments.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Allow auto-draft-pr to write' directly references the main change: adding write permissions to the auto-draft-pr GitHub Actions workflow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

@Gatsik Gatsik marked this pull request as ready for review December 20, 2025 11:54
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4dd9fdb and a0f2505.

📒 Files selected for processing (1)
  • .github/workflows/auto-draft-pr.yaml (2 hunks)

Copy link
Contributor

@BlackYps BlackYps left a comment

Choose a reason for hiding this comment

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

AI confidently being wrong again, lol.
Thank you for investigating and fixing this!

@BlackYps BlackYps merged commit 76c0348 into FAForever:develop Dec 27, 2025
4 of 5 checks passed
@BlackYps BlackYps changed the title Allow auto-draft-pr to write Fix permissions to auto-draft all PRs Dec 27, 2025
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