From 0bef636b04c1aa6873436e115a75489046c4b770 Mon Sep 17 00:00:00 2001
From: Daniel Stone <daniels@collabora.com>
Date: Sat, 31 Jul 2021 15:11:35 +0100
Subject: [PATCH] ci: Switch to using new rules syntax

The rules syntax is more clear and explicit than the previous
only/except. Switch to using that.

Signed-off-by: Daniel Stone <daniels@collabora.com>
---
 .gitlab-ci.yml | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 674ba243..c6c1573e 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -24,9 +24,14 @@ stages:
     FDO_DISTRIBUTION_EXEC: 'bash .gitlab-ci/debian-install.sh'
     FDO_DISTRIBUTION_TAG: '2021-07-31.0-pipewire'
 
+.ci-rules:
+  rules:
+    - when: on_success
+
 
 container_prep:
   extends:
+    - .ci-rules
     - .debian
     - .fdo.container-build@debian
   timeout: 30m
@@ -35,6 +40,7 @@ container_prep:
 
 .build-env:
   extends:
+    - .ci-rules
     - .debian
     - .fdo.distribution-image@debian
   timeout: 5m
@@ -140,6 +146,7 @@ build-no-gl:
 
 docs-and-coverage:
   extends:
+    - .ci-rules
     - .debian
     - .fdo.distribution-image@debian
   stage: pages
@@ -160,6 +167,7 @@ docs-and-coverage:
     - Documentation/
     - Test_Coverage/
 
+# does not inherit .ci-rules
 pages:
   extends:
     - .debian
@@ -175,5 +183,7 @@ pages:
   artifacts:
     paths:
     - public
-  only:
-    - main
+  rules:
+    - if: '$CI_PIPELINE_SOURCE == "push" && $CI_PROJECT_PATH == "wayland/weston" && $CI_BUILD_REF_NAME == $CI_DEFAULT_BRANCH'
+      when: on_success
+    - when: never