CodelyTV / pr-size-labeler

🏷 Visualize and optionally limit the size of your Pull Requests
https://github.com/marketplace/actions/pull-request-size-labeler
MIT License
336 stars 58 forks source link

Match the whole label with grep #65

Closed lionseal closed 5 months ago

lionseal commented 8 months ago

Hello!

I added -w option to grep so it matches the whole label name.

We have other labels in our repos and it's matching the first character, ie:

When grep runs it matches Simulation, thus later removing it from the PR (which we don't want!)

johnlk commented 5 months ago

Interesting, not sure why there's not a full word match from the beginning. I suppose if the labels are "size/xs" as the defaults of this project are, then a partial match is usually fine because it's unlikely to conflict with other labels.

lionseal commented 5 months ago

Thanks for reviewing! 🚀