Skip to content

Add the PR size check - #12

Merged
ryanbarlow97 merged 1 commit into
mainfrom
pr-size-gate
Oct 11, 2026
Merged

ryanbarlow97 merged 1 commit into
mainfrom
pr-size-gate

Conversation

@ryanbarlow97

Copy link
Copy Markdown
Contributor

Adds the shared PR size check from TF-Minecraft/TLibs#39. It labels pull requests size:XS to size:XXL and, from size:L, requires an approval from someone other than the author. See Pull request size.

After this merges, the ruleset requires the "PR size" check instead of one approval on every pull request.

🤖 Generated with Claude Code

Labels pull requests by size and requires a second approval from size:L.
See https://github.com/TF-Minecraft/Docs/blob/main/PIPELINES.md#pull-request-size

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added the size:S 10-29 effective changed lines (tests excluded in mixed PRs). label Oct 11, 2026
@coderabbitai

coderabbitai Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 0fa52101-7ebf-4a6a-80a7-30650271c96c

📥 Commits

Reviewing files that changed from the base of the PR and between b6c5905 and ec7c478.


📒 Files selected for processing (1)
  • .github/workflows/pr-size.yml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.



📝 Summary

Summary by CodeRabbit

  • New Features
    • Pull requests are now automatically assessed for size during key review and update events, making their scope easier to gauge.
    • The workflow manages overlapping checks for the same pull request and completes within a set time limit.

Walkthrough

Adds a GitHub Actions workflow that runs a pinned PR-size action for pull request lifecycle events and submitted or dismissed reviews.

Changes

Pull request size workflow

Layer / File(s) Summary
Workflow triggers and execution
.github/workflows/pr-size.yml
The workflow defines pull request and review event triggers, permissions, and per-pull-request concurrency cancellation. Its job checks out full history without persisted credentials, then runs the pinned PR-size action on Ubuntu 24.04 with a five-minute timeout.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Feature

Sequence Diagram(s)

sequenceDiagram
  participant PullRequestEvent as GitHub pull request or review event
  participant Workflow as GitHub Actions workflow
  participant Checkout as actions/checkout
  participant PRSize as Pinned PR-size action
  PullRequestEvent->>Workflow: Trigger workflow
  Workflow->>Checkout: Check out full history without persisted credentials
  Workflow->>PRSize: Run PR-size action
Loading

Merge Risk: ⚪ Minimal · up to ec7c4

The PR size check appears ready to merge after normal workflow checks.

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@ryanbarlow97
ryanbarlow97 merged commit b9c7e0c into main Oct 11, 2026
4 checks passed
@ryanbarlow97
ryanbarlow97 deleted the pr-size-gate branch October 11, 2026 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 effective changed lines (tests excluded in mixed PRs).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant