You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by hu...@apache.org on 2022/01/14 20:17:56 UTC

[superset] 01/01: Skip CI for frontend test

This is an automated email from the ASF dual-hosted git repository.

hugh pushed a commit to branch ci-skip-docs
in repository https://gitbox.apache.org/repos/asf/superset.git

commit 91dc80b51e60d3069f4950e1298191cdacd9e38a
Author: Hugh A. Miles II <hu...@gmail.com>
AuthorDate: Fri Jan 14 15:16:33 2022 -0500

    Skip CI for frontend test
---
 .github/workflows/superset-frontend.yml | 25 +++++++++++++++++--------
 1 file changed, 17 insertions(+), 8 deletions(-)

diff --git a/.github/workflows/superset-frontend.yml b/.github/workflows/superset-frontend.yml
index 03e2b9e..f689cf5 100644
--- a/.github/workflows/superset-frontend.yml
+++ b/.github/workflows/superset-frontend.yml
@@ -13,14 +13,23 @@ jobs:
     if: github.event.pull_request.draft == false
     runs-on: ubuntu-20.04
     steps:
+      - uses: dorny/paths-filter@v2
+        id: filter
+        with:
+          filters: |
+            docs:
+                - 'docs/**'
       - name: "Checkout ${{ github.ref }} ( ${{ github.sha }} )"
+        if: steps.changes.outputs.docs == 'false'
         uses: actions/checkout@v2
         with:
           persist-credentials: false
           submodules: recursive
       - name: Check npm lock file version
+        if: steps.changes.outputs.docs == 'true'
         run: ./scripts/ci_check_npm_lock_version.sh ./superset-frontend/package-lock.json
       - name: Check if frontend changes are present
+        if: steps.changes.outputs.docs == 'true'
         id: check
         env:
           GITHUB_REPO: ${{ github.repository }}
@@ -28,40 +37,40 @@ jobs:
         continue-on-error: true
         run: ./scripts/ci_check_no_file_changes.sh frontend
       - name: Setup Node.js
-        if: steps.check.outcome == 'failure'
+        if: steps.check.outcome == 'failure' || steps.changes.outputs.docs == 'true'
         uses: actions/setup-node@v2
         with:
           node-version: "16"
       - name: Install dependencies
-        if: steps.check.outcome == 'failure'
+        if: steps.check.outcome == 'failure' || steps.changes.outputs.docs == 'true'
         uses: ./.github/actions/cached-dependencies
         with:
           run: npm-install
       - name: lint
-        if: steps.check.outcome == 'failure'
+        if: steps.check.outcome == 'failure' || steps.changes.outputs.docs == 'true'
         working-directory: ./superset-frontend
         run: |
           npm run lint
           npm run prettier-check
       - name: Build plugins packages
-        if: steps.check.outcome == 'failure'
+        if: steps.check.outcome == 'failure' || steps.changes.outputs.docs == 'true'
         working-directory: ./superset-frontend
         run: npm run plugins:build
       - name: Build plugins Storybook
-        if: steps.check.outcome == 'failure'
+        if: steps.check.outcome == 'failure' || steps.changes.outputs.docs == 'true'
         working-directory: ./superset-frontend
         run: npm run plugins:build-storybook
       - name: unit tests
-        if: steps.check.outcome == 'failure'
+        if: steps.check.outcome == 'failure' || steps.changes.outputs.docs == 'true'
         working-directory: ./superset-frontend
         run: |
           npm run test -- --coverage
       # todo: remove this step when fix generator as a project in root jest.config.js
       - name: generator-superset unit tests
-        if: steps.check.outcome == 'failure'
+        if: steps.check.outcome == 'failure' || steps.changes.outputs.docs == 'true'
         working-directory: ./superset-frontend/packages/generator-superset
         run: npx jest
       - name: Upload code coverage
-        if: steps.check.outcome == 'failure'
+        if: steps.check.outcome == 'failure' || steps.changes.outputs.docs == 'true'
         working-directory: ./superset-frontend
         run: ../.github/workflows/codecov.sh -c -F javascript