Skip to content

Make the PR size check labels only - #30

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

ryanbarlow97 merged 1 commit into
mainfrom
pr-size-labels-only

Conversation

@ryanbarlow97

Copy link
Copy Markdown
Contributor

Moves the PR size check to TF-Minecraft/TLibs#40, which only applies the size:XS–size:XXL label and no longer needs an approval from size:L. Reviews no longer re-run the check.

🤖 Generated with Claude Code

@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: 297b3ed2-2451-4fc4-b21a-4167d2de8c9c

📥 Commits

Reviewing files that changed from the base of the PR and between 143ce22 and c88d9cf.


📒 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

  • Chores
    • Pull request size checks no longer run when a pull request review is submitted. They continue to run for the other configured pull request activities.

Walkthrough

The PR-size workflow no longer runs on pull request review events. The PR-size action reference now points to a different commit.

Changes

PR-size workflow

Layer / File(s) Summary
Update PR-size workflow
.github/workflows/pr-size.yml
The workflow removes the pull_request_review trigger and its submitted and dismissed activity types. The PR-size action reference changes to commit 0bfb259256648c6139c6a94a0935492830cbbec9.

Priority: ⬇️ Low

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

Change: Feature


Merge Risk: ⚪ Minimal · up to c88d9

Size labels continue to be applied without an approval gate, and reviews no longer rerun the workflow. No actionable merge risk remains.

  • 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.

@github-actions github-actions Bot added the size:XS 0-9 effective changed lines (tests excluded in mixed PRs). label Oct 11, 2026
@ryanbarlow97
ryanbarlow97 merged commit 05c6b10 into main Oct 11, 2026
3 checks passed
@ryanbarlow97
ryanbarlow97 deleted the pr-size-labels-only branch October 11, 2026 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 0-9 effective changed lines (tests excluded in mixed PRs).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant