ci: Bring back job skipping for PRs without code changes

Author: Lms24

.github/workflows/build.yml

@@ -96,7 +96,8 @@ jobs:
             profiling_node:
               - 'packages/profiling-node/**'
               - 'dev-packages/e2e-tests/test-applications/node-profiling/**'
-
+            any_code:
+              - '!**/*.md'
 
       - name: Get PR labels
         id: pr-labels
@@ -109,6 +110,7 @@ jobs:
       is_release: ${{ startsWith(github.ref, 'refs/heads/release/') }}
       changed_profiling_node: ${{ steps.changed.outputs.profiling_node == 'true' }}
       changed_ci: ${{ steps.changed.outputs.workflow == 'true' }}
+      changed_any_code: ${{ steps.changed.outputs.any_code == 'true' }}
       # When merging into master, or from master
       is_gitflow_sync: ${{ github.head_ref == 'master' || github.ref == 'refs/heads/master' }}
       has_gitflow_label:
@@ -158,6 +160,10 @@ jobs:
         if: steps.cache_dependencies.outputs.cache-hit != 'true'
         run: yarn install --ignore-engines --frozen-lockfile
 
+      - name: Skip step if no code changes
+        if: needs.job_get_metadata.outputs.changed_any_code == 'false'
+        run: echo "No code changes, skipping build."
+
       - name: Check for Affected Nx Projects
         uses: dkhunt27/[email protected]
         id: checkForAffected
@@ -204,6 +210,12 @@ jobs:
       changed_browser_integration: ${{ needs.job_get_metadata.outputs.changed_ci == 'true' || contains(steps.checkForAffected.outputs.affected, '@sentry-internal/browser-integration-tests') }}
       # If you are looking for changed_profiling_node, this is defined in job_get_metadata
 
+  job_changed_any_code:
+    name: Check for code changes
+    needs: [job_get_metadata, job_build]
+    runs-on: ubuntu-20.04
+    if: needs.job_get_metadata.outputs.changed_any_code == 'true'
+
   job_check_branches:
     name: Check PR branches
     needs: job_get_metadata
@@ -221,7 +233,7 @@ jobs:
 
   job_size_check:
     name: Size Check
-    needs: [job_get_metadata, job_build]
+    needs: [job_get_metadata, job_build, job_changed_any_code]
     timeout-minutes: 15
     runs-on: ubuntu-20.04
     if:
@@ -251,7 +263,7 @@ jobs:
     name: Lint
     # Even though the linter only checks source code, not built code, it needs the built code in order check that all
     # inter-package dependencies resolve cleanly.
-    needs: [job_get_metadata, job_build]
+    needs: [job_get_metadata, job_build, job_changed_any_code]
     timeout-minutes: 10
     runs-on: ubuntu-20.04
     steps:
@@ -297,7 +309,7 @@ jobs:
 
   job_circular_dep_check:
     name: Circular Dependency Check
-    needs: [job_get_metadata, job_build]
+    needs: [job_get_metadata, job_build, job_changed_any_code]
     timeout-minutes: 10
     runs-on: ubuntu-20.04
     steps:
@@ -318,7 +330,7 @@ jobs:
 
   job_artifacts:
     name: Upload Artifacts
-    needs: [job_get_metadata, job_build, job_compile_bindings_profiling_node]
+    needs: [job_get_metadata, job_build, job_changed_any_code, job_compile_bindings_profiling_node]
     runs-on: ubuntu-20.04
     # Build artifacts are only needed for releasing workflow.
     if: needs.job_get_metadata.outputs.is_release == 'true'
@@ -360,7 +372,7 @@ jobs:
 
   job_browser_unit_tests:
     name: Browser Unit Tests
-    needs: [job_get_metadata, job_build]
+    needs: [job_get_metadata, job_build, job_changed_any_code]
     timeout-minutes: 10
     runs-on: ubuntu-20.04
     steps:
@@ -398,7 +410,7 @@ jobs:
 
   job_bun_unit_tests:
     name: Bun Unit Tests
-    needs: [job_get_metadata, job_build]
+    needs: [job_get_metadata, job_build, job_changed_any_code]
     if: needs.job_build.outputs.changed_bun == 'true' || github.event_name != 'pull_request'
     timeout-minutes: 10
     runs-on: ubuntu-20.04
@@ -425,7 +437,7 @@ jobs:
 
   job_deno_unit_tests:
     name: Deno Unit Tests
-    needs: [job_get_metadata, job_build]
+    needs: [job_get_metadata, job_build, job_changed_any_code]
     if: needs.job_build.outputs.changed_deno == 'true' || github.event_name != 'pull_request'
     timeout-minutes: 10
     runs-on: ubuntu-20.04
@@ -456,7 +468,7 @@ jobs:
 
   job_node_unit_tests:
     name: Node (${{ matrix.node }}) Unit Tests
-    needs: [job_get_metadata, job_build]
+    needs: [job_get_metadata, job_build, job_changed_any_code]
     timeout-minutes: 10
     runs-on: ubuntu-20.04
     strategy:
@@ -501,7 +513,7 @@ jobs:
 
   job_profiling_node_unit_tests:
     name: Node Profiling Unit Tests
-    needs: [job_get_metadata, job_build]
+    needs: [job_get_metadata, job_build, job_changed_any_code]
     if: needs.job_build.outputs.changed_node == 'true' || needs.job_get_metadata.outputs.changed_profiling_node == 'true' || github.event_name != 'pull_request'
     runs-on: ubuntu-latest
     timeout-minutes: 10
@@ -529,7 +541,7 @@ jobs:
 
   job_browser_playwright_tests:
     name: Playwright ${{ matrix.bundle }}${{ matrix.project && matrix.project != 'chromium' && format(' {0}', matrix.project) || ''}}${{ matrix.shard && format(' ({0}/{1})', matrix.shard, matrix.shards) || ''}} Tests
-    needs: [job_get_metadata, job_build]
+    needs: [job_get_metadata, job_build, job_changed_any_code]
     if: needs.job_build.outputs.changed_browser_integration == 'true' || github.event_name != 'pull_request'
     runs-on: ubuntu-20.04-large-js
     timeout-minutes: 25
@@ -609,7 +621,7 @@ jobs:
 
   job_browser_loader_tests:
     name: PW ${{ matrix.bundle }} Tests
-    needs: [job_get_metadata, job_build]
+    needs: [job_get_metadata, job_build, job_changed_any_code]
     if: needs.job_build.outputs.changed_browser_integration == 'true' || github.event_name != 'pull_request'
     runs-on: ubuntu-20.04
     timeout-minutes: 15
@@ -659,7 +671,7 @@ jobs:
 
   job_check_for_faulty_dts:
     name: Check for faulty .d.ts files
-    needs: [job_get_metadata, job_build]
+    needs: [job_get_metadata, job_build, job_changed_any_code]
     runs-on: ubuntu-20.04
     timeout-minutes: 5
     steps:
@@ -686,7 +698,7 @@ jobs:
     name:
       Node (${{ matrix.node }})${{ (matrix.typescript && format(' (TS {0})', matrix.typescript)) || '' }} Integration
       Tests
-    needs: [job_get_metadata, job_build]
+    needs: [job_get_metadata, job_build, job_changed_any_code]
     if: needs.job_build.outputs.changed_node_integration == 'true' || github.event_name != 'pull_request'
     runs-on: ubuntu-20.04
     timeout-minutes: 15
@@ -728,7 +740,7 @@ jobs:
 
   job_remix_integration_tests:
     name: Remix v${{ matrix.remix }} (Node ${{ matrix.node }}) Tests
-    needs: [job_get_metadata, job_build]
+    needs: [job_get_metadata, job_build, job_changed_any_code]
     if: needs.job_build.outputs.changed_remix == 'true' || github.event_name != 'pull_request'
     runs-on: ubuntu-20.04
     timeout-minutes: 10
@@ -776,7 +788,7 @@ jobs:
     if: |
       always() && needs.job_build.result == 'success' &&
       (needs.job_compile_bindings_profiling_node.result == 'success' || needs.job_compile_bindings_profiling_node.result == 'skipped')
-    needs: [job_get_metadata, job_build, job_compile_bindings_profiling_node]
+    needs: [job_get_metadata, job_build, job_changed_any_code, job_compile_bindings_profiling_node]
     runs-on: ubuntu-20.04-large-js
     timeout-minutes: 15
     steps:
@@ -834,7 +846,7 @@ jobs:
     # We need to add the `always()` check here because the previous step has this as well :(
     # See: https://github.com/actions/runner/issues/2205
     if: always() && needs.job_e2e_prepare.result == 'success'
-    needs: [job_get_metadata, job_build, job_e2e_prepare]
+    needs: [job_get_metadata, job_build, job_changed_any_code, job_e2e_prepare]
     runs-on: ubuntu-20.04
     timeout-minutes: 10
     env:
@@ -999,7 +1011,7 @@ jobs:
       always() && needs.job_e2e_prepare.result == 'success' &&
       (github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name == github.repository) &&
       github.actor != 'dependabot[bot]'
-    needs: [job_get_metadata, job_build, job_e2e_prepare]
+    needs: [job_get_metadata, job_build, job_changed_any_code, job_e2e_prepare]
     runs-on: ubuntu-20.04
     timeout-minutes: 10
     env:
@@ -1114,7 +1126,7 @@ jobs:
         (needs.job_get_metadata.outputs.is_release == 'true') ||
         (github.event_name != 'pull_request')
       )
-    needs: [job_get_metadata, job_build, job_e2e_prepare]
+    needs: [job_get_metadata, job_build, job_changed_any_code, job_e2e_prepare]
     runs-on: ubuntu-20.04
     timeout-minutes: 10
     env:
@@ -1224,7 +1236,7 @@ jobs:
 
   overhead_metrics:
     name: Overhead metrics
-    needs: [job_get_metadata, job_build]
+    needs: [job_get_metadata, job_build, job_changed_any_code]
     runs-on: ubuntu-20.04
     timeout-minutes: 30
     if: |
@@ -1265,7 +1277,7 @@ jobs:
 
   job_compile_bindings_profiling_node:
     name: Compile & Test Profiling Bindings (v${{ matrix.node }}) ${{ matrix.target_platform || matrix.os }}, ${{ matrix.node || matrix.container }}, ${{ matrix.arch || matrix.container }}, ${{ contains(matrix.container, 'alpine') && 'musl' || 'glibc'  }}
-    needs: [job_get_metadata, job_build]
+    needs: [job_get_metadata, job_build, job_changed_any_code]
     # Compiling bindings can be very slow (especially on windows), so only run precompile
     # Skip precompile unless we are on a release branch as precompile slows down CI times.
     if: |