From c6fd9091846d568e9129bf5f9e125ea9591165d0 Mon Sep 17 00:00:00 2001 From: "Anna (cybertailor) Vyalkova" Date: Sun, 23 Apr 2023 20:11:11 +0500 Subject: [PATCH] .github/workflows/emails.yml: unbroke Signed-off-by: Anna (cybertailor) Vyalkova --- .github/workflows/emails.yml | 3 +-- scripts/email-checker.py | 6 ++++-- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/.github/workflows/emails.yml b/.github/workflows/emails.yml index 1594833282..67763cdb92 100644 --- a/.github/workflows/emails.yml +++ b/.github/workflows/emails.yml @@ -27,9 +27,8 @@ jobs: run: | COMMIT=$(gh run list -R ${GITHUB_REPOSITORY} --branch=dev --workflow=emails.yml --status=success --limit=1 --json=headSha --jq=".[0].headSha") echo "last_successful_commit=${COMMIT}" >> "${GITHUB_ENV}" - echo "Workflow last succeeded on commit ${COMMIT:0:7}" - if git rev-parse --verify "${COMMIT}^{commit}"; then + if git rev-parse --verify "${COMMIT}^{commit}" > /dev/null; then echo -e "\033[92m\u2713 ${COMMIT:0:7} points to a valid commit" VALID=true else diff --git a/scripts/email-checker.py b/scripts/email-checker.py index e0e4779132..aa23550e53 100755 --- a/scripts/email-checker.py +++ b/scripts/email-checker.py @@ -47,5 +47,7 @@ def check_maintainers(maintainers: Iterator[Maintainer]) -> Iterator[Maintainer] if __name__ == '__main__': - missing_maintainers = len(tuple(check_maintainers(set(read_all_maintainers(sys.argv[1:]))))) - sys.exit(int(missing_maintainers != 0)) + files = [line.strip() for line in sys.stdin] + maintainers = set(read_all_maintainers(files)) + missing_maintainers = tuple(check_maintainers(maintainers)) + sys.exit(int(len(missing_maintainers) != 0))