From 6eda4722f66652203a53cb4277f46efb331ca22a Mon Sep 17 00:00:00 2001 From: Alex Ivanov Date: Wed, 31 Jul 2024 23:03:46 +0100 Subject: [PATCH 1/2] Move Meticulous job to a separate workflow --- .github/workflows/ci.yaml | 20 --------------- .github/workflows/meticulous.yaml | 42 +++++++++++++++++++++++++++++++ 2 files changed, 42 insertions(+), 20 deletions(-) create mode 100644 .github/workflows/meticulous.yaml diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index 4b665be19afd1..8c71a06880a3e 100644 --- a/.github/workflows/ci.yaml +++ b/.github/workflows/ci.yaml @@ -1005,23 +1005,3 @@ jobs: fi done echo "No incompatible licenses detected" - meticulous: - runs-on: ubuntu-latest - steps: - - name: "Checkout Repository" - uses: actions/checkout@v4 - - name: Setup Node - uses: ./.github/actions/setup-node - - name: Build - working-directory: ./site - run: pnpm build - - name: Serve - working-directory: ./site - run: | - pnpm vite preview & - sleep 5 - - name: Run Meticulous tests - uses: alwaysmeticulous/report-diffs-action/cloud-compute@v1 - with: - api-token: ${{ secrets.METICULOUS_API_TOKEN }} - app-url: "http://127.0.0.1:4173/" diff --git a/.github/workflows/meticulous.yaml b/.github/workflows/meticulous.yaml new file mode 100644 index 0000000000000..f9adbf4577add --- /dev/null +++ b/.github/workflows/meticulous.yaml @@ -0,0 +1,42 @@ +# Workflow for serving the webapp locally & running Meticulous tests against it. + +name: Meticulous + +on: + push: + branches: + - main + pull_request: + # Meticulous needs the workflow to be triggered on workflow_dispatch events, + # so that Meticulous can run the workflow on the base commit to compare + # against if an existing workflow hasn't run. + workflow_dispatch: + +permissions: + actions: write + contents: read + issues: write + pull-requests: write + statuses: read + +jobs: + meticulous: + runs-on: ubuntu-latest + steps: + - name: "Checkout Repository" + uses: actions/checkout@v4 + - name: Setup Node + uses: ./.github/actions/setup-node + - name: Build + working-directory: ./site + run: pnpm build + - name: Serve + working-directory: ./site + run: | + pnpm vite preview & + sleep 5 + - name: Run Meticulous tests + uses: alwaysmeticulous/report-diffs-action/cloud-compute@v1 + with: + api-token: ${{ secrets.METICULOUS_API_TOKEN }} + app-url: "http://127.0.0.1:4173/" From fc846de96e5f9d7bea5d5dd71d2c45db042559cb Mon Sep 17 00:00:00 2001 From: Alex Ivanov Date: Wed, 31 Jul 2024 23:05:27 +0100 Subject: [PATCH 2/2] Run Meticulous only on changes to site/2 --- .github/workflows/meticulous.yaml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/.github/workflows/meticulous.yaml b/.github/workflows/meticulous.yaml index f9adbf4577add..b1542858e7490 100644 --- a/.github/workflows/meticulous.yaml +++ b/.github/workflows/meticulous.yaml @@ -6,7 +6,11 @@ on: push: branches: - main + paths: + - "site/**" pull_request: + paths: + - "site/**" # Meticulous needs the workflow to be triggered on workflow_dispatch events, # so that Meticulous can run the workflow on the base commit to compare # against if an existing workflow hasn't run.