Merge pull request #11 from jerseysu/fix-pipeline-2

ci(playwright.yaml): add lint check to flow

Author: jerseysu

.github/workflows/pages-build-deployment.yml

@@ -1,13 +1,29 @@
 name: Publish
 on:
-  push:
-    branches: [main]
+  workflow_run:
+    workflows: ['Playwright-Tests']
+    types:
+      - completed
 
 jobs:
   deployment:
     timeout-minutes: 60
     name: "deploy to github"
     runs-on: ubuntu-latest
+    if: github.event_name != 'pull_request'
     steps:
-      - name: deploy to github page
-        run: npm run deploy
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: lts/*
+      - name: Install dependencies
+        run: npm ci
+      - name: Lint
+        run: npm run lint
+      - name: Build
+        run: npm run build
+      - name: Deploy to Github Pages
+        uses: peaceiris/actions-gh-pages@v3
+        with:
+          github_token: ${{ secrets.GITHUB_TOKEN }}
+          publish_dir: build

.github/workflows/playwright.yml

@@ -1,4 +1,4 @@
-name: Playwright Tests
+name: Playwright-Tests
 on:
   push:
     branches: [main]
@@ -16,6 +16,8 @@ jobs:
           node-version: lts/*
       - name: Install dependencies
         run: npm ci
+      - name: Lint
+        run: npm run lint
       - name: Install Playwright Browsers
         run: npx playwright install --with-deps
       - name: Run Playwright tests

package.json

@@ -29,6 +29,7 @@
     "build": "react-scripts build",
     "test": "react-scripts test",
     "test:e2e": "npx playwright test",
+    "lint": "npx eslint .",
     "eject": "react-scripts eject"
   },
   "eslintConfig": {