From 076a09917d1ea159e93b108a21e3c99e18ebe268 Mon Sep 17 00:00:00 2001
From: William Desportes <williamdes@wdes.fr>
Date: Mon, 28 Mar 2022 19:09:41 +0200
Subject: [PATCH] Remove non needed anymore ci skip code on workflows

Ref: https://github.blog/changelog/2021-02-08-github-actions-skip-pull-request-and-push-workflows-with-skip-ci/

Signed-off-by: William Desportes <williamdes@wdes.fr>
---
 .github/workflows/lint-and-analyse-php.yml | 3 ---
 .github/workflows/lint-docs.yml            | 1 -
 .github/workflows/other-tools.yml          | 2 --
 .github/workflows/test-selenium.yml        | 2 +-
 .github/workflows/tests.yml                | 5 -----
 5 files changed, 1 insertion(+), 12 deletions(-)

diff --git a/.github/workflows/lint-and-analyse-php.yml b/.github/workflows/lint-and-analyse-php.yml
index 72814d8f11..7215291307 100644
--- a/.github/workflows/lint-and-analyse-php.yml
+++ b/.github/workflows/lint-and-analyse-php.yml
@@ -11,7 +11,6 @@ on:
 jobs:
     lint-node:
         runs-on: ubuntu-latest
-        if: "!contains(github.event.head_commit.message, '[ci skip]')"
         steps:
             - uses: actions/checkout@v2
             - uses: actions/setup-node@v1
@@ -36,7 +35,6 @@ jobs:
 
     lint-php-files:
         runs-on: ubuntu-latest
-        if: "!contains(github.event.head_commit.message, '[ci skip]')"
         steps:
             - uses: actions/checkout@v2
             - name: Use php 7.1
@@ -72,7 +70,6 @@ jobs:
 
     analyse-php:
         runs-on: ubuntu-latest
-        if: "!contains(github.event.head_commit.message, '[ci skip]')"
         steps:
             - uses: actions/checkout@v2
             - name: Use php 7.2
diff --git a/.github/workflows/lint-docs.yml b/.github/workflows/lint-docs.yml
index 36c706f435..dac27f8263 100644
--- a/.github/workflows/lint-docs.yml
+++ b/.github/workflows/lint-docs.yml
@@ -11,7 +11,6 @@ on:
 jobs:
     lint-docs:
         runs-on: ubuntu-latest
-        if: "!contains(github.event.head_commit.message, '[ci skip]')"
         steps:
             - uses: actions/checkout@v2
             - name: Lint phpdoc blocks
diff --git a/.github/workflows/other-tools.yml b/.github/workflows/other-tools.yml
index 7405593415..c7550a335f 100644
--- a/.github/workflows/other-tools.yml
+++ b/.github/workflows/other-tools.yml
@@ -11,7 +11,6 @@ on:
 jobs:
     build-documentation:
         runs-on: ubuntu-latest
-        if: "!contains(github.event.head_commit.message, '[ci skip]')"
         steps:
             - uses: actions/checkout@v2
             - uses: actions/setup-python@v2
@@ -24,7 +23,6 @@ jobs:
 
     build-release:
         runs-on: ubuntu-latest
-        if: "!contains(github.event.head_commit.message, '[ci skip]')"
         steps:
             - uses: actions/checkout@v2
             - name: Install gettext
diff --git a/.github/workflows/test-selenium.yml b/.github/workflows/test-selenium.yml
index dc5cd6d0ce..45eed9d11f 100644
--- a/.github/workflows/test-selenium.yml
+++ b/.github/workflows/test-selenium.yml
@@ -11,7 +11,7 @@ on:
 jobs:
     test-selenium:
         name: Selenium tests on php ${{ matrix.php-version }} and ${{ matrix.os }}
-        if: "!contains(github.event.head_commit.message, '[ci skip]') && !contains(github.event.head_commit.message, '[ci selenium skip]')"
+        if: "!contains(github.event.head_commit.message, '[ci selenium skip]')"
         runs-on: ${{ matrix.os }}
         services:
           database-server:
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index a1e672f845..ce7f8622de 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -11,7 +11,6 @@ on:
 jobs:
     multi-arch-tests-php:
         name: Test on php ${{ matrix.php-version }} (${{ matrix.arch }})
-        if: "!contains(github.event.head_commit.message, '[ci skip]')"
         runs-on: ubuntu-latest
         continue-on-error: ${{ matrix.experimental }}
         strategy:
@@ -60,7 +59,6 @@ jobs:
                 name: phpunit-${{ matrix.php-version }}-${{ matrix.arch }}
     test-php:
         name: Test on php ${{ matrix.php-version }} and ${{ matrix.os }}
-        if: "!contains(github.event.head_commit.message, '[ci skip]')"
         runs-on: ${{ matrix.os }}
         continue-on-error: ${{ matrix.experimental }}
         strategy:
@@ -123,7 +121,6 @@ jobs:
     test-php-optional-extensions:
         name: Test on php (+ ext-${{ matrix.extension }}) ${{ matrix.php-version }} and ${{ matrix.os }}
         runs-on: ${{ matrix.os }}
-        if: "!contains(github.event.head_commit.message, '[ci skip]')"
         strategy:
             matrix:
                 php-version: ["7.1"]
@@ -182,7 +179,6 @@ jobs:
     test-php-random:
         name: Test (random) on php ${{ matrix.php-version }} and ${{ matrix.os }}
         runs-on: ${{ matrix.os }}
-        if: "!contains(github.event.head_commit.message, '[ci skip]')"
         strategy:
             matrix:
                 php-version: ["7.4"]
@@ -218,7 +214,6 @@ jobs:
     test-js:
         name: Test javascript files
         runs-on: ubuntu-latest
-        if: "!contains(github.event.head_commit.message, '[ci skip]')"
         steps:
             - uses: actions/checkout@v2
             - uses: actions/setup-node@v1