Skip to content

Add PR size labeling workflow#1255

Open
mhucka wants to merge 7 commits intoquantumlib:mainfrom
mhucka:add-size-labeler
Open

Add PR size labeling workflow#1255
mhucka wants to merge 7 commits intoquantumlib:mainfrom
mhucka:add-size-labeler

Conversation

@mhucka
Copy link
Copy Markdown
Contributor

@mhucka mhucka commented Apr 3, 2026

This workflow is the same as what we use in Cirq to label PRs with size: S, size: M, etc., to indicate the size of the changes.

This workflow is identical to the one we use in Cirq. It labels PRs with
`size: S`, `size: M`, etc., to indicate the size of the changes.
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a Bash script, size-labeler.sh, designed to automatically categorize pull requests by size (XS to XL) based on the number of lines changed, excluding specific file patterns. Feedback focuses on addressing a limitation where only the first 100 files are processed by implementing API pagination, ensuring consistent capitalization across all size labels, and optimizing JSON processing by removing unnecessary temporary file creation in the jq helper functions.

mhucka added 3 commits April 3, 2026 02:08
This enhances the code to support getting more than 100 files from
GitHub.

(Written with the help of Gemini CLI.)
@mhucka
Copy link
Copy Markdown
Contributor Author

mhucka commented Apr 3, 2026

/gemini review

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a new CI toolset, including a README and a bash script (size-labeler.sh) designed to automatically assign size labels (XS to XL) to pull requests based on the number of lines changed, excluding specific generated or lock files. A review comment identified a casing inconsistency in the 'Size: XS' label which would cause issues with the script's logic for pruning stale labels.

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.

1 participant