Do not early return when no relevant code owner exists

[saschanaz]

Fixes #39

I'd like to have a test but this doesn't really fit into the existing unit tests.

[saschanaz]

The tests pass locally, what does it mean by unexpected token? How can it happen? 🤔

[orta]

Hah, no idea about that jest error - maybe a weird npx issue?

I think I'd prefer if the label code just moves up above the return instead, mainly so that it's still really obvious that it's not going to accidentally do any work in the fn if no users are found

[saschanaz]

I think I'd prefer if the label code just moves up above the return instead, mainly so that it's still really obvious that it's not going to accidentally do any work in the fn if no users are found

Isn't it obvious as the code still returns inside if (!ownersWhoHaveAccessToAllFilesInPR.length)?

[orta]

👍🏻