openshift-helm-charts / development

0 stars 17 forks source link

Correctly determine the OWNERS file change type #340

Closed komish closed 4 months ago

komish commented 4 months ago

Fixed #336

Per the original issue, this PR corrects an issue determining if the submitted OWNERS file was modified or added.

I originally suggested in the issue that the comment should not be posted if the OWNERS file is being modified, but I changed my mind, and think it may be worth leaving so that a maintainer has more information available to them when they decide to merge an OWNERS file modification for a Red Hat or Community submission.

If this ternary syntax looks odd, check it out in action in my test repo where I run both sides of the ternary expression.