From d8e302bbfb63d30f6771830a3d2e89badfffac2c Mon Sep 17 00:00:00 2001
From: Sviatoslav Sydorenko <wk@sydorenko.org.ua>
Date: Tue, 4 Jul 2023 19:11:57 +0200
Subject: [PATCH] =?UTF-8?q?=F0=9F=8E=A8=F0=9F=94=A7=F0=9F=91=B7=20Wire=20i?=
 =?UTF-8?q?n=20linters=20into=20the=20main=20workflow?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This patch makes use of reusable workflows to embed the linting jobs
into the main pipeline. The technique allows having modularized CI
definitions stored in separate files but being voting in the
alls-green gate used in branch protection.
---
 .github/workflows/ci.yml                      |  5 +++++
 .github/workflows/{qa.yml => reusable-qa.yml} | 10 +---------
 2 files changed, 6 insertions(+), 9 deletions(-)
 rename .github/workflows/{qa.yml => reusable-qa.yml} (89%)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 220087b84..d58113a4b 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -38,6 +38,10 @@ env:
     PRE_COMMIT_COLOR
 
 jobs:
+  linters:
+    name: Linters
+    uses: ./.github/workflows/reusable-qa.yml
+
   test:
     name: ${{ matrix.os }} / ${{ matrix.python-version }} / ${{ matrix.pip-version }}
     runs-on: ${{ matrix.os }}-latest
@@ -181,6 +185,7 @@ jobs:
     if: always()
 
     needs:
+      - linters
       - pypy
       - test
 
diff --git a/.github/workflows/qa.yml b/.github/workflows/reusable-qa.yml
similarity index 89%
rename from .github/workflows/qa.yml
rename to .github/workflows/reusable-qa.yml
index a88600165..9525a3920 100644
--- a/.github/workflows/qa.yml
+++ b/.github/workflows/reusable-qa.yml
@@ -1,15 +1,7 @@
 name: QA
 
 on:
-  pull_request:
-  push:
-    branches:
-      - main
-    tags:
-
-concurrency:
-  group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}
-  cancel-in-progress: true
+  workflow_call:
 
 jobs:
   qa: