Skip to content

Conversation

@jvbriones
Copy link
Contributor

@jvbriones jvbriones commented Aug 8, 2025

Adapt PR size job to add sizes label:

default values
size-XS: ≤ 10 lines
size-S: ≤ 100 lines
size-M: ≤ 500 lines
size-L: ≤ 1000 lines
size-XL: > 1000 lines

Also adapt the calculation of changes by adding +0 in the awk END block to ensure that when there are no changes, awk outputs 0 instead of an empty string:

{print add} → {print add+0}
{print del} → {print del+0}

tested here: https://github.com/MetaMask/metamask-mobile/actions/runs/16826192546/job/47663091866

@metamaskbot metamaskbot added the team-qa team-qa label Aug 8, 2025
cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

Copy link
Contributor

@mcmire mcmire left a comment

Choose a reason for hiding this comment

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

Seems right to me!

@jvbriones jvbriones merged commit f5f175e into main Aug 9, 2025
19 checks passed
@jvbriones jvbriones deleted the ci-pr-line-check branch August 9, 2025 04:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-qa team-qa

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants