Closed AlvaroGG0 closed 1 month ago
@AlvaroGG0 Looks good to me, thank you for the contribution!
@AlvaroGG0 I invited you as a collaborator to the repository. It seems you are using this action a lot more than I am at the moment. Would appreciate your help here.
The current verification of patterns does not follow the same logic as Github's when they do not start with
/
, which causes Codeowners to mismatch in these cases.From Github documentation:
Changes made:
**/
at the beginning of patterns in case they don't start with/
to follow the same logic./
is concatenated at the beginning of eachchangedFile
string to make the minimatch function work well on those patterns with a/
at the beginning."Required codeowners: {"$CODEOWNER":false}"
to"Required codeowners: $CODEOWNER"
as it can lead to confusion