chore: fix some workflows (#3254)

* add conditions to lint check * lint * delete cosmoverse * conditional * try integration * fix * fix * attempt test * integration * docs deploy * md-lint fix * check testdata changes for tests * remove md-link-checker * re-add link check
ignite · Dec 9, 2022 · 597e70f · 597e70f
1 parent 1f84fcf
commit 597e70f
Show file tree

Hide file tree

Showing 7 changed files with 58 additions and 75 deletions.
diff --git a/.github/workflows/docs-deploy-preview.yml b/.github/workflows/docs-deploy-preview.yml
@@ -12,23 +12,31 @@ jobs:
 
     steps:
       - name: Checkout
-        uses: actions/checkout@v3
+        uses: actions/checkout@v3.1.0
+
+      - uses: technote-space/get-diff-action@v6.1.1
+        with:
+          PATTERNS: |
+            ./docs/**.md
 
       - name: Cache node_modules
         uses: actions/cache@v3
+        if: env.GIT_DIFF
         with:
           path: docs/node_modules
           key: node_modules-${{ hashFiles('**/yarn.lock') }}
 
       - name: Setup node
         uses: actions/setup-node@v3
+        if: env.GIT_DIFF
         with:
           node-version: 16.x
           cache: yarn
           cache-dependency-path: docs/yarn.lock
 
       - name: Install Dependencies
         run: yarn install
+        if: env.GIT_DIFF
         working-directory: ./docs
 
       - name: Build
@@ -37,6 +45,7 @@ jobs:
 
       - name: Deploy
         uses: FirebaseExtended/action-hosting-deploy@v0
+        if: env.GIT_DIFF
         with:
           repoToken: "${{ secrets.GITHUB_TOKEN }}"
           firebaseServiceAccount: "${{ secrets.FIREBASE_SERVICE_ACCOUNT_IGNTSERVICES }}"

diff --git a/.github/workflows/md-link-checker-config.json b/.github/workflows/md-link-checker-config.json
@@ -28,4 +28,4 @@
   "retryOn429": true,
   "retryCount": 5,
   "fallbackRetryDelay": "30s"
-}
+}
diff --git a/.github/workflows/md-link-checker.yml b/.github/workflows/md-link-checker.yml
@@ -1,12 +1,20 @@
 name: Check Markdown links
-on: push
+on:
+  push:
+    branches:
+      - main
 jobs:
   markdown-link-check:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3.1.0
+      - uses: technote-space/get-diff-action@v6.1.1
+        with:
+          PATTERNS: |
+            **/*.md
       - uses: gaurav-nelson/github-action-markdown-link-check@1.0.13
+        if: env.GIT_DIFF
         with:
           folder-path: "."
           use-verbose-mode: 'yes'
-          config-file: '.github/workflows/md-link-checker-config.json'
+          config-file: '.github/workflows/md-link-checker-config.json'
diff --git a/.github/workflows/release-cosmoverse.yml b/.github/workflows/release-cosmoverse.yml
diff --git a/.github/workflows/test-integration.yml b/.github/workflows/test-integration.yml
@@ -13,7 +13,7 @@ jobs:
     outputs:
       matrix: ${{ steps.set-matrix.outputs.matrix }}
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v3.1.0
       - name: Finding files and store to output
         id: set-matrix
         run: echo "matrix=$({ cd integration && find . -type d ! -name testdata -maxdepth 1 -print; } | tail -n +2 | cut -c 3- | jq -R . | jq -cs .)" >> $GITHUB_OUTPUT
@@ -29,8 +29,17 @@ jobs:
       matrix:
         test-path: ${{fromJson(needs.pre-test.outputs.matrix)}}
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v3.1.0
+      - uses: technote-space/get-diff-action@v6.1.1
+        with:
+          PATTERNS: |
+            **/*.go
+            go.mod
+            go.sum
+            **/testdata/**
+
       - uses: actions/cache@v3
+        if: env.GIT_DIFF
         with:
           path: |
             ~/.cache/go-build
@@ -39,9 +48,11 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-go-
       - uses: actions/setup-go@v3
+        if: env.GIT_DIFF
         with:
           go-version: 1.19
       - name: Run Integration Tests
+        if: env.GIT_DIFF
         run: GOSUMDB=off go test -v -timeout 60m ./integration/${{ matrix.test-path }}
 
   status:

diff --git a/.github/workflows/test-lint.yml b/.github/workflows/test-lint.yml
@@ -12,12 +12,19 @@ jobs:
     runs-on: ubuntu-latest
     timeout-minutes: 6
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v3.1.0
+      - uses: technote-space/get-diff-action@v6.1.1
+        with:
+          PATTERNS: |
+            **/*.go
+            go.mod
+            go.sum
+
       - name: Set up Go
         uses: actions/setup-go@v3
         with:
           go-version: 1.19
-      - uses: golangci/golangci-lint-action@master
+      - uses: golangci/golangci-lint-action@v3
         with:
           # Required: the version of golangci-lint is required and must be specified without patch version: we always use the latest patch version.
           version: v1.50.1

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -11,16 +11,30 @@ jobs:
   test:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v3.1.0
+      - uses: technote-space/get-diff-action@v6.1.1
+        with:
+          PATTERNS: |
+            **/*.go
+            go.mod
+            go.sum
+            **/testdata/**
 
       - uses: actions/setup-go@v3
+        if: env.GIT_DIFF
         with:
           go-version: 1.19
+          cache: true
+          cache-dependency-path: go.sum
+
       - run: ./scripts/test-coverage
+        if: env.GIT_DIFF
 
       - name: Upload coverage to Codecov
         uses: codecov/codecov-action@v3.1.1
+        if: env.GIT_DIFF
         with:
           file: ./coverage.txt
           fail_ci_if_error: true
           verbose: true
+