Merge pull request #657 from llm-d-incubation/copilot/skip-e2e-tests-on-no-code-changes

Skip e2e tests for documentation-only PRs in ci-pr-checks workflow

Author: asm582

.github/workflows/ci-pr-checks.yaml

@@ -7,6 +7,30 @@ on:
       - dev
 
 jobs:
+  # Check if PR contains code changes (not just docs/metadata)
+  check-code-changes:
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+    outputs:
+      has_code_changes: ${{ steps.filter.outputs.code }}
+    steps:
+      - name: Checkout source
+        uses: actions/checkout@v4
+      
+      - name: Check for code changes
+        uses: dorny/paths-filter@v3
+        id: filter
+        with:
+          filters: |
+            code:
+              - '!docs/**'
+              - '!README.md'
+              - '!CONTRIBUTING.md'
+              - '!LICENSE'
+              - '!OWNERS'
+              - '!PROJECT'
+
   lint-and-test:
     runs-on: ubuntu-latest
     steps:
@@ -47,9 +71,11 @@ jobs:
   # E2E tests run in parallel with different configurations:
   # - HPAScaleToZero feature gate settings (true/false)
   # - GPU types (nvidia/amd) for limiter tests
+  # Skip e2e tests if PR only contains docs/metadata changes
   e2e-tests:
     runs-on: ubuntu-latest
-    needs: lint-and-test
+    needs: [lint-and-test, check-code-changes]
+    if: needs.check-code-changes.outputs.has_code_changes == 'true'
     timeout-minutes: 60
     strategy:
       fail-fast: false

.gitignore

@@ -34,3 +34,4 @@ llmd/
 llmd-infra/
 
 *.tgz
+actionlint