[tracker/wip/carlosg/ci-playground: 1/2] ci: Add review stage

[Carlos Garnacho <carlosg src gnome org>]

commit 5ab73a1ce954fa4fec2b05f6eceb0c0913d3d7d5
Author: Carlos Garnacho <carlosg gnome org>
Date:   Sun May 16 15:54:47 2021 +0200

    ci: Add review stage
    
    And add checks for commit logs, and ensuring that merge requests
    have "Allow edits from maintainers" checkbox enabled. Courtesy of
    ci-fairy.

 .gitlab-ci.yml | 56 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 56 insertions(+)
---
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 81b7e60d4..4ba7267e2 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -14,12 +14,53 @@ variables:
   MESON_TEST_EXTRA_ARGS: ""
 
 stages:
+  - review
   - prepare
   - build
   - test
   - analysis
   - website
 
+.check-template: &check
+  extends:
+    - .fdo.ci-fairy
+  artifacts:
+    expire_in: 1 week
+    paths:
+      - check-junit-report.xml
+    reports:
+      junit: check-junit-report.xml
+
+check-commit-log:
+  variables:
+    GIT_DEPTH: "100"
+  stage: review
+  script:
+    - if [[ x"$CI_MERGE_REQUEST_TARGET_BRANCH_NAME" != "x" ]] ;
+      then
+        ci-fairy check-commits --junit-xml=check-junit-report.xml ;
+      else
+        echo "Not a merge request" ;
+      fi
+  only:
+    - merge_requests
+  <<: *check
+
+check-merge-request:
+  variables:
+    GIT_STRATEGY: none
+  stage: review
+  script:
+    - if [[ x"$CI_MERGE_REQUEST_TARGET_BRANCH_NAME" != "x" ]] ;
+      then
+        ci-fairy check-merge-request --require-allow-collaboration --junit-xml=check-junit-report.xml ;
+      else
+        echo "Not a merge request" ;
+      fi
+  only:
+    - merge_requests
+  <<: *check
+
 .tracker.fedora@common:
   variables:
     BASE_TAG: '2021-05-15.8'
@@ -87,6 +128,9 @@ build-fedora-container@x86_64:
   stage: prepare
   variables:
     GIT_STRATEGY: none
+  needs:
+    - check-commit-log
+    - check-merge-request
 
 build-fedora-rawhide-container@x86_64:
   extends:
@@ -95,6 +139,9 @@ build-fedora-rawhide-container@x86_64:
   stage: prepare
   variables:
     GIT_STRATEGY: none
+  needs:
+    - check-commit-log
+    - check-merge-request
 
 build-fedora-container@aarch64:
   extends:
@@ -103,6 +150,9 @@ build-fedora-container@aarch64:
   stage: prepare
   variables:
     GIT_STRATEGY: none
+  needs:
+    - check-commit-log
+    - check-merge-request
 
 build-ubuntu-container@x86_64:
   extends:
@@ -111,6 +161,9 @@ build-ubuntu-container@x86_64:
   stage: prepare
   variables:
     GIT_STRATEGY: none
+  needs:
+    - check-commit-log
+    - check-merge-request
 
 build-alpine-container@x86_64:
   extends:
@@ -119,6 +172,9 @@ build-alpine-container@x86_64:
   stage: prepare
   variables:
     GIT_STRATEGY: none
+  needs:
+    - check-commit-log
+    - check-merge-request
 
 .build-template: &build
   stage: build