Skip to content

Commit 69bc2e9

Browse files
committed
run compile only on release or if pkg changed
1 parent 2bbfe75 commit 69bc2e9

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
@@ -542,7 +542,7 @@ jobs:
542542
job_profiling_node_unit_tests:
543543
name: Node Profiling Unit Tests
544544
needs: [job_get_metadata, job_build]
545-
if: needs.job_get_metadata.outputs.changed_profiling_node == 'true' || github.event_name != 'pull_request'
545+
if: needs.job_get_metadata.outputs.changed_node || needs.job_get_metadata.outputs.changed_profiling_node == 'true' || github.event_name != 'pull_request'
546546
runs-on: ubuntu-latest
547547
timeout-minutes: 10
548548
steps:
@@ -1212,6 +1212,12 @@ jobs:
12121212
job_compile_bindings_profiling_node:
12131213
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' }}
12141214
needs: [job_get_metadata, job_install_deps, job_build]
1215+
# Compiling bindings can be very slow (especially on windows), so only run precompile
1216+
# if profiling or profiling node package had changed or if we are on a release branch.
1217+
if: |
1218+
(needs.job_get_metadata.outputs.changed_profiling_node == 'true') ||
1219+
(needs.job_get_metadata.outputs.is_release) ||
1220+
(github.event_name != 'pull_request')
12151221
runs-on: ${{ matrix.os }}
12161222
container: ${{ matrix.container }}
12171223
timeout-minutes: 30

0 commit comments

Comments
 (0)