fix(ci): comment skip code for required workflows (#2312)

ChainSafe · Feb 23, 2022 · 45dce9b · 45dce9b
1 parent 9284dc7
commit 45dce9b
Show file tree

Hide file tree

Showing 13 changed files with 165 additions and 137 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -3,22 +3,24 @@ on:
   push:
     branches:
       - development
-    paths:
-      - .github/workflows/build.yml
-      - "**/*.go"
-      - "chain/**"
-      - "cmd/**"
-      - "dot/**"
-      - "internal/**"
-      - "lib/**"
-      - "pkg/**"
-      - "tests/**"
-      - .dockerignore
-      - .codecov.yml
-      - Dockerfile
-      - go.mod
-      - go.sum
-      - Makefile
+    # Commented paths to avoid skipping required workflow
+    # See https://github.community/t/feature-request-conditional-required-checks/16761
+    # paths:
+    #   - .github/workflows/build.yml
+    #   - "**/*.go"
+    #   - "chain/**"
+    #   - "cmd/**"
+    #   - "dot/**"
+    #   - "internal/**"
+    #   - "lib/**"
+    #   - "pkg/**"
+    #   - "tests/**"
+    #   - .dockerignore
+    #   - .codecov.yml
+    #   - Dockerfile
+    #   - go.mod
+    #   - go.sum
+    #   - Makefile
 
 jobs:
   builds:

diff --git a/.github/workflows/checks.yml b/.github/workflows/checks.yml
@@ -1,12 +1,14 @@
 on:
   pull_request:
-    paths:
-      - .github/workflows/checks.yml
-      - .github/PULL_REQUEST/pull_request.go
-      - .golangci.yml
-      - "**/*.go"
-      - go.mod
-      - go.sum
+    # Commented paths to avoid skipping required workflow
+    # See https://github.community/t/feature-request-conditional-required-checks/16761
+    # paths:
+    #   - .github/workflows/checks.yml
+    #   - .github/PULL_REQUEST/pull_request.go
+    #   - .golangci.yml
+    #   - "**/*.go"
+    #   - go.mod
+    #   - go.sum
 
 name: checks
 env:
@@ -27,7 +29,9 @@ jobs:
 
   check-description:
     name: Checks PR has title and body description
-    if: ${{ github.actor != 'dependabot[bot]' }}
+    # Commented to avoid skipping required workflow
+    # See https://github.community/t/feature-request-conditional-required-checks/16761
+    # if: ${{ github.actor != 'dependabot[bot]' }}
     runs-on: ubuntu-latest
     steps:
       - uses: actions/setup-go@v2

diff --git a/.github/workflows/code-cov.yml b/.github/workflows/code-cov.yml
@@ -1,18 +1,20 @@
 on:
   pull_request:
-    paths:
-      - .github/workflows/code-cov.yml
-      - "**/*.go"
-      - "chain/**"
-      - "cmd/**"
-      - "dot/**"
-      - "internal/**"
-      - "lib/**"
-      - "pkg/**"
-      - "tests/**"
-      - .codecov.yml
-      - go.mod
-      - go.sum
+    # Commented paths to avoid skipping required workflow
+    # See https://github.community/t/feature-request-conditional-required-checks/16761
+    # paths:
+    #   - .github/workflows/code-cov.yml
+    #   - "**/*.go"
+    #   - "chain/**"
+    #   - "cmd/**"
+    #   - "dot/**"
+    #   - "internal/**"
+    #   - "lib/**"
+    #   - "pkg/**"
+    #   - "tests/**"
+    #   - .codecov.yml
+    #   - go.mod
+    #   - go.sum
 name: code-cov
 env:
   GO111MODULE: on

diff --git a/.github/workflows/copyright.yml b/.github/workflows/copyright.yml
@@ -3,10 +3,12 @@ on:
   pull_request:
     branches:
       - development
-    paths:
-      - .github/workflows/copyright.yml
-      - "**/*.go"
-      - "**/*.proto"
+    # Commented paths to avoid skipping required workflow
+    # See https://github.community/t/feature-request-conditional-required-checks/16761
+    # paths:
+    #   - .github/workflows/copyright.yml
+    #   - "**/*.go"
+    #   - "**/*.proto"
 
 jobs:
   copyright-check:

diff --git a/.github/workflows/docker-grandpa.yml b/.github/workflows/docker-grandpa.yml
@@ -1,17 +1,19 @@
 on:
   pull_request:
-    paths:
-      - .github/workflows/docker-grandpa.yml
-      - "**/*.go"
-      - "chain/**"
-      - "cmd/**"
-      - "dot/**"
-      - "internal/**"
-      - "lib/**"
-      - "pkg/**"
-      - "tests/stress/**"
-      - go.mod
-      - go.sum
+    # Commented paths to avoid skipping required workflow
+    # See https://github.community/t/feature-request-conditional-required-checks/16761
+    # paths:
+    #   - .github/workflows/docker-grandpa.yml
+    #   - "**/*.go"
+    #   - "chain/**"
+    #   - "cmd/**"
+    #   - "dot/**"
+    #   - "internal/**"
+    #   - "lib/**"
+    #   - "pkg/**"
+    #   - "tests/stress/**"
+    #   - go.mod
+    #   - go.sum
 name: docker-grandpa
 env:
   GO111MODULE: on

diff --git a/.github/workflows/docker-js.yml b/.github/workflows/docker-js.yml
@@ -1,17 +1,19 @@
 on:
   pull_request:
-    paths:
-      - .github/workflows/docker-js.yml
-      - "**/*.go"
-      - "chain/**"
-      - "cmd/**"
-      - "dot/**"
-      - "internal/**"
-      - "lib/**"
-      - "pkg/**"
-      - "tests/polkadotjs_test/**"
-      - go.mod
-      - go.sum
+    # Commented paths to avoid skipping required workflow
+    # See https://github.community/t/feature-request-conditional-required-checks/16761
+    # paths:
+    #   - .github/workflows/docker-js.yml
+    #   - "**/*.go"
+    #   - "chain/**"
+    #   - "cmd/**"
+    #   - "dot/**"
+    #   - "internal/**"
+    #   - "lib/**"
+    #   - "pkg/**"
+    #   - "tests/polkadotjs_test/**"
+    #   - go.mod
+    #   - go.sum
 name: docker-js
 env:
   GO111MODULE: on

diff --git a/.github/workflows/docker-rpc.yml b/.github/workflows/docker-rpc.yml
@@ -1,17 +1,19 @@
 on:
   pull_request:
-    paths:
-      - .github/workflows/docker-rpc.yml
-      - "**/*.go"
-      - "chain/**"
-      - "cmd/**"
-      - "dot/**"
-      - "internal/**"
-      - "lib/**"
-      - "pkg/**"
-      - "tests/rpc/**"
-      - go.mod
-      - go.sum
+    # Commented paths to avoid skipping required workflow
+    # See https://github.community/t/feature-request-conditional-required-checks/16761
+    # paths:
+    #   - .github/workflows/docker-rpc.yml
+    #   - "**/*.go"
+    #   - "chain/**"
+    #   - "cmd/**"
+    #   - "dot/**"
+    #   - "internal/**"
+    #   - "lib/**"
+    #   - "pkg/**"
+    #   - "tests/rpc/**"
+    #   - go.mod
+    #   - go.sum
 name: docker-rpc
 env:
   GO111MODULE: on

diff --git a/.github/workflows/docker-stable.yml b/.github/workflows/docker-stable.yml
@@ -1,17 +1,19 @@
 on:
   pull_request:
-    paths:
-      - .github/workflows/docker-stable.yml
-      - "**/*.go"
-      - "chain/**"
-      - "cmd/**"
-      - "dot/**"
-      - "internal/**"
-      - "lib/**"
-      - "pkg/**"
-      - scripts/integration-test-all.sh
-      - go.mod
-      - go.sum
+    # Commented paths to avoid skipping required workflow
+    # See https://github.community/t/feature-request-conditional-required-checks/16761
+    # paths:
+    #   - .github/workflows/docker-stable.yml
+    #   - "**/*.go"
+    #   - "chain/**"
+    #   - "cmd/**"
+    #   - "dot/**"
+    #   - "internal/**"
+    #   - "lib/**"
+    #   - "pkg/**"
+    #   - scripts/integration-test-all.sh
+    #   - go.mod
+    #   - go.sum
 name: docker-stable
 env:
   GO111MODULE: on

diff --git a/.github/workflows/docker-stress.yml b/.github/workflows/docker-stress.yml
@@ -1,17 +1,19 @@
 on:
   pull_request:
-    paths:
-      - .github/workflows/docker-stress.yml
-      - "**/*.go"
-      - "chain/**"
-      - "cmd/**"
-      - "dot/**"
-      - "internal/**"
-      - "lib/**"
-      - "pkg/**"
-      - "tests/stress/**"
-      - go.mod
-      - go.sum
+    # Commented paths to avoid skipping required workflow
+    # See https://github.community/t/feature-request-conditional-required-checks/16761
+    # paths:
+    #   - .github/workflows/docker-stress.yml
+    #   - "**/*.go"
+    #   - "chain/**"
+    #   - "cmd/**"
+    #   - "dot/**"
+    #   - "internal/**"
+    #   - "lib/**"
+    #   - "pkg/**"
+    #   - "tests/stress/**"
+    #   - go.mod
+    #   - go.sum
 name: docker-stress
 env:
   GO111MODULE: on

diff --git a/.github/workflows/docs.yml b/.github/workflows/docs.yml
@@ -3,9 +3,11 @@ on:
   push:
     branches:
       - development
-    paths:
-      - .github/workflows/docs.yml
-      - "docs/**"
+    # Commented paths to avoid skipping required workflow
+    # See https://github.community/t/feature-request-conditional-required-checks/16761
+    # paths:
+    #   - .github/workflows/docs.yml
+    #   - "docs/**"
 
 jobs:
   build:

diff --git a/.github/workflows/integration-tests.yml b/.github/workflows/integration-tests.yml
@@ -1,28 +1,31 @@
 on:
   pull_request:
-    paths:
-      - .github/workflows/integration-tests.yml
-      - "**/*.go"
-      - "chain/**"
-      - "cmd/**"
-      - "dot/**"
-      - "internal/**"
-      - "lib/**"
-      - "pkg/**"
-      - "tests/**"
-      - go.mod
-      - go.sum
+    # Commented paths to avoid skipping required workflow
+    # See https://github.community/t/feature-request-conditional-required-checks/16761
+    # paths:
+    #   - .github/workflows/integration-tests.yml
+    #   - "**/*.go"
+    #   - "chain/**"
+    #   - "cmd/**"
+    #   - "dot/**"
+    #   - "internal/**"
+    #   - "lib/**"
+    #   - "pkg/**"
+    #   - "tests/**"
+    #   - go.mod
+    #   - go.sum
 name: integration-tests
 
 jobs:
   integration-tests:
     timeout-minutes: 60
     strategy:
       matrix:
-        packages: [
-          github.com/ChainSafe/gossamer/dot/rpc/modules,
-          github.com/ChainSafe/gossamer/lib/babe
-        ]
+        packages:
+          [
+            github.com/ChainSafe/gossamer/dot/rpc/modules,
+            github.com/ChainSafe/gossamer/lib/babe,
+          ]
     runs-on: ubuntu-latest
     steps:
       - id: go-cache-paths
@@ -53,4 +56,3 @@ jobs:
 
       - name: Run integration tests
         run: go test -timeout=30m -tags integration ${{ matrix.packages }}
-
diff --git a/.github/workflows/mocks.yml b/.github/workflows/mocks.yml
@@ -3,9 +3,11 @@ on:
   pull_request:
     branches:
       - development
-    paths:
-      - .github/workflows/mocks.yml
-      - "**/*.go"
+    # Commented paths to avoid skipping required workflow
+    # See https://github.community/t/feature-request-conditional-required-checks/16761
+    # paths:
+    #   - .github/workflows/mocks.yml
+    #   - "**/*.go"
 
 jobs:
   mocks-check: