Conversation
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.
There was a problem hiding this comment.
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.
This enhances the code to support getting more than 100 files from GitHub. (Written with the help of Gemini CLI.)
|
/gemini review |
There was a problem hiding this comment.
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.
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.