Decompose the pr testing workflow

Author: dshevtsov

.github/workflows/lint-pr_job.yml

@@ -0,0 +1,48 @@
+name: Reusable workflow to lint a pull request
+
+on: workflow_call
+
+jobs:
+    lint-test:
+        runs-on: ubuntu-latest
+
+        steps:
+            - name: Checkout Code
+              uses: actions/checkout@v4
+              with:
+                  fetch-depth: 0
+
+            - name: Setup reviewdog
+              uses: reviewdog/action-setup@v1.3.0
+              with:
+                  reviewdog_version: v0.20.3
+
+            - name: Load super-linter configuration
+              shell: bash
+              run: cat .github/super-linter.env >> "$GITHUB_ENV"
+
+            - name: Lint Code Base with super-linter and reviewdog
+              uses: super-linter/super-linter/slim@v7.3.0
+              env:
+                  GITHUB_TOKEN: ${{ github.token }}
+
+            - name: Report linting results with reviewdog
+              shell: bash
+              run: |
+                  TMPFILE=$(mktemp)
+                  git diff > "${TMPFILE}"
+                  if [ ! -s "${TMPFILE}" ]; then
+                      echo "No changes detected, skipping reviewdog."
+                      exit 0
+                  fi
+                  git stash --include-untracked
+                  reviewdog                        \
+                      -f=diff                        \
+                      -f.diff.strip=1                \
+                      -name="markdownlint-fix"       \
+                      -reporter="github-pr-review"   \
+                      -filter-mode="diff_context"    \
+                      -fail-level=error < "${TMPFILE}"
+                  git stash drop || true
+              env:
+                  REVIEWDOG_GITHUB_API_TOKEN: ${{ github.token }}

.github/workflows/lint-pull-request.yml

@@ -0,0 +1,9 @@
+
+name: Lint pull request
+
+on: pull_request_target
+
+jobs:
+  linting:
+    name: Check code with linters
+    uses: ./.github/workflows/lint-pr_job.yml

.github/workflows/test-pull-request.yml

@@ -1,6 +1,7 @@
 name: Validate pull request
 
-on: pull_request_target
+on: pull_request
+
 permissions:
   contents: read
   statuses: write

.github/workflows/validate-pr_job.yml

@@ -1,83 +1,31 @@
----
 name: Reusable workflow to validate a pull request
 
 on: workflow_call
 
 jobs:
-  lint-test:
+  remark-test:
     runs-on: ubuntu-latest
-
     steps:
-    - name: Checkout Code
-      uses: actions/checkout@v4
-      with:
-        fetch-depth: 0
-
-    - name: Setup reviewdog
-      uses: reviewdog/action-setup@v1.3.0
-      with:
-        reviewdog_version: v0.20.3
-
-    - name: Load super-linter configuration
-      shell: bash
-      run: cat .github/super-linter.env >> "$GITHUB_ENV"
-
-    - name: Lint Code Base with super-linter and reviewdog
-      uses: super-linter/super-linter/slim@v7.3.0
-      env:
-        GITHUB_TOKEN: ${{ github.token }}
-
-    - name: Report linting results with reviewdog
-      shell: bash
-      run: |
-        TMPFILE=$(mktemp)
-        git diff > "${TMPFILE}"
-        if [ ! -s "${TMPFILE}" ]; then
-          echo "No changes detected, skipping reviewdog."
-          exit 0
-        fi
-        git stash --include-untracked
-        reviewdog                        \
-          -f=diff                        \
-          -f.diff.strip=1                \
-          -name="markdownlint-fix"       \
-          -reporter="github-pr-review"   \
-          -filter-mode="diff_context"    \
-          -fail-level=error < "${TMPFILE}"
-        git stash drop || true
-      env:
-        REVIEWDOG_GITHUB_API_TOKEN: ${{ github.token }}
+      - uses: actions/checkout@v4
+      - name: Use Setup Node and Install Dependencies Action
+        uses: commerce-docs/devsite-install-action@main
+        with:
+          node-version-file: '.nvmrc'
+          cache-dependency-path: 'yarn.lock'
 
-  # remark-test:
-  #   permissions:
-  #     contents: read
-  #     statuses: write
-  #   runs-on: ubuntu-latest
-  #   needs: lint-test
-  #   steps:
-  #     - uses: actions/checkout@v4
-  #     - name: Use Setup Node and Install Dependencies Action
-  #       uses: commerce-docs/devsite-install-action@main
-  #       with:
-  #         node-version-file: '.nvmrc'
-  #         cache-dependency-path: 'yarn.lock'
+      - name: Check links
+        run: yarn test
 
-  #     - name: Check links
-  #       run: yarn test
-
-  # build-test:
-  #   permissions:
-  #     contents: read
-  #     statuses: write
-  #   runs-on: ubuntu-latest
-  #   needs: remark-test
-  #   steps:
-  #     - uses: actions/checkout@v4
-  #     - name: Use Setup Node and Install Dependencies Action
-  #       uses: commerce-docs/devsite-install-action@main
-  #       with:
-  #         node-version-file: '.nvmrc'
-  #         cache-dependency-path: 'yarn.lock'
-
-  #     - name: Build site
-  #       run: yarn build
+  build-test:
+    runs-on: ubuntu-latest
+    needs: remark-test
+    steps:
+      - uses: actions/checkout@v4
+      - name: Use Setup Node and Install Dependencies Action
+        uses: commerce-docs/devsite-install-action@main
+        with:
+          node-version-file: '.nvmrc'
+          cache-dependency-path: 'yarn.lock'
+
+      - name: Build site
+        run: yarn build