Skip to content

Commit a813479

Browse files
committed
run compile only on release or if pkg changed
1 parent 3fed38c commit a813479

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

.github/workflows/build.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -515,7 +515,7 @@ jobs:
515515
job_profiling_node_unit_tests:
516516
name: Node Profiling Unit Tests
517517
needs: [job_get_metadata, job_build]
518-
if: needs.job_get_metadata.outputs.changed_profiling_node == 'true' || github.event_name != 'pull_request'
518+
if: needs.job_get_metadata.outputs.changed_node || needs.job_get_metadata.outputs.changed_profiling_node == 'true' || github.event_name != 'pull_request'
519519
runs-on: ubuntu-latest
520520
timeout-minutes: 10
521521
steps:
@@ -1125,6 +1125,12 @@ jobs:
11251125
job_compile_bindings_profiling_node:
11261126
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' }}
11271127
needs: [job_get_metadata, job_install_deps, job_build]
1128+
# Compiling bindings can be very slow (especially on windows), so only run precompile
1129+
# if profiling or profiling node package had changed or if we are on a release branch.
1130+
if: |
1131+
(needs.job_get_metadata.outputs.changed_profiling_node == 'true') ||
1132+
(needs.job_get_metadata.outputs.is_release) ||
1133+
(github.event_name != 'pull_request')
11281134
runs-on: ${{ matrix.os }}
11291135
container: ${{ matrix.container }}
11301136
timeout-minutes: 30

0 commit comments

Comments
 (0)