Skip to content

Make the PR size check labels only - #28

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

@github-actions github-actions Bot added the size:XS 0-9 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: ba29b909-a733-47a6-86e1-e345007c34c9

📥 Commits

Reviewing files that changed from the base of the PR and between d935061 and fb7b842.


📒 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 review is submitted or dismissed. They continue to run for the workflow’s other configured events.
    • This change affects when size checks are triggered; it does not alter the reported size information or the review process.

Walkthrough

The PR-size workflow no longer runs when pull request reviews are submitted or dismissed. It also uses a different revision of the PR-size action.

Changes

PR-size workflow

Layer / File(s) Summary
Workflow triggers and action revision
.github/workflows/pr-size.yml
Removed the pull_request_review events for submitted and dismissed reviews. Updated the PR-size action revision to 0bfb259256648c6139c6a94a0935492830cbbec9.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other


Merge Risk: ⚪ Minimal · up to fb7b8

Size labels continue to be applied without requiring approval for size L, and reviews no longer rerun the check. This matches the stated behavior, with no identified merge-blocking risk.

  • 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 069e210 into main Oct 11, 2026
3 checks passed
@ryanbarlow97
ryanbarlow97 deleted the pr-size-labels-only branch October 11, 2026 17:35
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