Skip to content

Commit 8bcfbb3

Browse files
committed
ci: scope Java/JS E2Es to language-specific paths
Expand e2e_java and e2e_js change detection to include shared pipeline code (optimization/, verification/, languages/base.py) but decouple from the broad e2e flag. A change to codeflash/version.py now only triggers Python E2Es, not Java/JS E2Es.
1 parent 3d50ac5 commit 8bcfbb3

1 file changed

Lines changed: 13 additions & 8 deletions

File tree

.github/workflows/ci.yaml

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -74,13 +74,20 @@ jobs:
7474
check_paths e2e \
7575
'codeflash/' 'tests/' 'pyproject.toml' 'uv.lock'
7676
77-
# JS E2E tests: JS packages changed
77+
# JS E2E tests: JS language support + shared pipeline + packages
7878
check_paths e2e_js \
79-
'packages/'
79+
'codeflash/languages/javascript/' 'codeflash/languages/base.py' \
80+
'codeflash/languages/registry.py' 'codeflash/optimization/' \
81+
'codeflash/verification/' 'packages/' \
82+
'tests/' 'pyproject.toml' 'uv.lock'
8083
81-
# Java E2E tests: java runtime or java test fixtures changed
84+
# Java E2E tests: Java language support + shared pipeline + runtime
8285
check_paths e2e_java \
83-
'codeflash-java-runtime/' 'code_to_optimize/java/'
86+
'codeflash/languages/java/' 'codeflash/languages/base.py' \
87+
'codeflash/languages/registry.py' 'codeflash/optimization/' \
88+
'codeflash/verification/' 'codeflash-java-runtime/' \
89+
'code_to_optimize/java/' \
90+
'tests/' 'pyproject.toml' 'uv.lock'
8491
env:
8592
MERGE_BASE: ${{ steps.merge_base.outputs.sha }}
8693

@@ -313,8 +320,7 @@ jobs:
313320
e2e-js:
314321
needs: determine-changes
315322
if: >-
316-
(needs.determine-changes.outputs.e2e == 'true'
317-
|| needs.determine-changes.outputs.e2e_js == 'true')
323+
needs.determine-changes.outputs.e2e_js == 'true'
318324
&& github.event_name != 'push'
319325
strategy:
320326
fail-fast: false
@@ -399,8 +405,7 @@ jobs:
399405
e2e-java:
400406
needs: determine-changes
401407
if: >-
402-
(needs.determine-changes.outputs.e2e == 'true'
403-
|| needs.determine-changes.outputs.e2e_java == 'true')
408+
needs.determine-changes.outputs.e2e_java == 'true'
404409
&& github.event_name != 'push'
405410
strategy:
406411
fail-fast: false

0 commit comments

Comments
 (0)