From 2ffcc1aa3fe3ac37272d36ad0f0ae495f66f0804 Mon Sep 17 00:00:00 2001 From: Usama Sadiq Date: Thu, 24 Aug 2023 22:38:52 +0500 Subject: [PATCH] build: distinguish unit test and pylint successful checks (#32452) We will be able to use these composite checks to block PR merges instead of blocking on every single individual unit test and pylint check. Closes: https://github.com/openedx/edx-platform/issues/32238 --- .github/workflows/pylint-checks.yml | 2 +- .github/workflows/unit-tests.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/pylint-checks.yml b/.github/workflows/pylint-checks.yml index 3c10098e4d..50f4bf6fcc 100644 --- a/.github/workflows/pylint-checks.yml +++ b/.github/workflows/pylint-checks.yml @@ -71,7 +71,7 @@ jobs: # https://github.com/marketplace/actions/alls-green#why # https://github.com/orgs/community/discussions/33579 success: - name: Tests successful + name: Pylint checks successful if: always() needs: - run-pylint diff --git a/.github/workflows/unit-tests.yml b/.github/workflows/unit-tests.yml index c66c30d7be..6ec56ec0b2 100644 --- a/.github/workflows/unit-tests.yml +++ b/.github/workflows/unit-tests.yml @@ -81,7 +81,7 @@ jobs: # https://github.com/marketplace/actions/alls-green#why # https://github.com/orgs/community/discussions/33579 success: - name: Tests successful + name: Unit tests successful if: (github.repository == 'edx/edx-platform-private') || (github.repository == 'openedx/edx-platform' && (startsWith(github.base_ref, 'open-release') == false)) needs: - run-tests