build: merge code lint and test both jobs in single workflow

wpmudev · Oct 10, 2023 · 17c5b82 · 17c5b82
1 parent 38d6772
commit 17c5b82
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 67 deletions.
diff --git a/.github/workflows/code-lint.yml → .github/workflows/code-review.yml b/.github/workflows/code-lint.yml → .github/workflows/code-review.yml
@@ -1,4 +1,4 @@
-name: Code linting
+name: SUI 3 Codes review
 
 on:
   pull_request:
@@ -11,7 +11,7 @@ on:
       - master
 
 jobs:
-  code-linting:
+  fetch-repos:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout sui-docs repository
@@ -24,12 +24,12 @@ jobs:
       - name: Grant execute permission for sui-docs
         run: chmod +x ./sui-docs
 
-#      - name: Checkout sui-css repository
-#        uses: actions/checkout@v3
-#        with:
-#          repository: wpmudev/sui-css
-#          path: ./sui-docs/packages/sui-css
-#          ref: development
+      #     - name: Checkout sui-css repository
+      #       uses: actions/checkout@v3
+      #       with:
+      #         repository: wpmudev/sui-css
+      #         path: ./sui-docs/packages/sui-css
+      #         ref: development
 
       - name: Checkout sui-icons repository
         uses: actions/checkout@v3
@@ -46,6 +46,10 @@ jobs:
       - name: List content
         run: pwd && ls
 
+  code-testing:
+    needs: [fetch-repos]
+    runs-on: ubuntu-latest
+    steps:
       - name: Set up Node.js
         uses: actions/setup-node@v3
         with:
@@ -54,6 +58,13 @@ jobs:
       - name: Install dependencies
         run: cd ./sui-docs && npm install --force
 
+      - name: Run tests
+        run: cd ./sui-docs && npm run test
+
+  code-linting:
+    needs: [fetch-repos]
+    runs-on: ubuntu-latest
+    steps:
       - name: Run eslint on changed files
         uses: tj-actions/eslint-changed-files@v21
         with:

diff --git a/.github/workflows/code-test.yml b/.github/workflows/code-test.yml