PRs from forks are missed by required-labels - Githubissues

dimagi / required-labels

🏷 Automated label checking for GitHub pull requests

BSD 3-Clause "New" or "Revised" License

47 stars 16 forks source link

PRs from forks are missed by required-labels #12

Closed dannyroberts closed 6 years ago

dannyroberts commented 6 years ago

@proteusvacuum

proteusvacuum commented 6 years ago

The PR in question (https://github.com/dimagi/commcare-hq/pull/19061/commits/87a0a667590f491e77502e8627a4e16d43acc7a2) is returning a 404 from required-labels.

https://github.com/dimagi/commcare-hq/settings/hooks/17560961 Can see delivery with ID e71c7af0-f494-11e7-8330-ce103811b736

I think the offending line of code is here: https://github.com/dimagi/required-labels/blob/6cf5d0cb0b0f920669386568ff1b5285c6eb6fff/utils.py#L42-L44 which returns "sophiamcg/commcare-hq" for this PR.