notowned with an empty codeowners file

[electriquo]

Description

Using version 0.7.1 with an empty CODEOWNERS file and setting experimental_checks: notowned displays an error and it does not fail the execution (see Actual result for the error message).

Expected result

If there is an error, the validator should fail with non zero exit code

Actual result

time="2022-02-27T09:21:01Z" level=error msg="while executing checker [Experimental] Not Owned File Checker: 1 error occurred:\n\t* fatal: No pathspec was given. Which files should I remove?\n: command \"xargs\": exit status 123\n\n" service="check:runner"

Steps to reproduce

Use the same setup as #121 with an empty CODEOWNERS file.

Troubleshooting

• Used the same setup with a CODEOWNERS with a single line * @org/team and the error did not appear

[mszostok]

It was a known issue:

codeowners-validator/internal/runner/runner_worker.go

Line 72 in d95ed83

// TODO(mszostok): add err handling (logging it internally is not enough)

But now it's fixed by #130