Skip to content

Commit c24ec97

Browse files
committed
ci: apply suggestions from code review
1 parent 9f70508 commit c24ec97

File tree

1 file changed

+7
-10
lines changed

1 file changed

+7
-10
lines changed

.github/workflows/CommonCI.yml

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -134,8 +134,6 @@ jobs:
134134
shell: julia --color=yes --code-coverage=user --depwarn=yes --project=. {0}
135135
# Only in Julia v1.10 we need to install `ReactantCore` manually.
136136
if: ${{ inputs.julia_version == '1.10' || inputs.julia_version == 'lts' }}
137-
env:
138-
JULIA_PKG_SERVER_REGISTRY_PREFERENCE: eager
139137

140138
# Run the tests (pjrt or ifrt or both)
141139
- name: "Setup Runtime Preferences (PJRT)"
@@ -160,10 +158,7 @@ jobs:
160158
shell: julia --color=yes --code-coverage=user --depwarn=yes --project=. {0}
161159
env:
162160
ALLOW_RERESOLVE: ${{ !inputs.downgrade_testing }}
163-
JULIA_PKG_SERVER_REGISTRY_PREFERENCE: eager
164161
REACTANT_TEST_GROUP: ${{ inputs.test_group }}
165-
XLA_FLAGS: "--xla_force_host_platform_device_count=12"
166-
JULIA_DEBUG: "Reactant,Reactant_jll"
167162

168163
- name: "Setup Runtime Preferences (IFRT)"
169164
if: ${{ inputs.runtime == 'ifrt' || inputs.runtime == 'both' }}
@@ -187,23 +182,20 @@ jobs:
187182
shell: julia --color=yes --code-coverage=user --depwarn=yes --project=. {0}
188183
env:
189184
ALLOW_RERESOLVE: ${{ !inputs.downgrade_testing }}
190-
JULIA_PKG_SERVER_REGISTRY_PREFERENCE: eager
191185
REACTANT_TEST_GROUP: ${{ inputs.test_group }}
192-
XLA_FLAGS: "--xla_force_host_platform_device_count=12"
193-
JULIA_DEBUG: "Reactant,Reactant_jll"
194186

195187
- name: "Upload MLIR modules"
196188
uses: actions/upload-artifact@v4
197189
timeout-minutes: 10
198-
if: ${{ always() }}
190+
if: always()
199191
with:
200192
name: "mlir-${{ inputs.julia_version }}-${{ inputs.os }}-${{ inputs.runtime }}-assertions=${{ inputs.assertions }}-${{ github.event_name }}-test_group=${{ inputs.test_group }}-${{ inputs.localjll }}"
201193
path: "**/*.mlir"
202194
retention-days: 90
203195
overwrite: false
204196

205197
- name: Save Julia depot cache on cancel or failure
206-
if: ${{ inputs.localjll && (cancelled() || failure()) }}
198+
if: cancelled() || failure()
207199
uses: actions/cache/save@v4
208200
with:
209201
path: |
@@ -215,3 +207,8 @@ jobs:
215207
with:
216208
files: lcov.info
217209
token: ${{ secrets.CODECOV_TOKEN }}
210+
211+
env:
212+
JULIA_PKG_SERVER_REGISTRY_PREFERENCE: eager
213+
XLA_FLAGS: "--xla_force_host_platform_device_count=12"
214+
JULIA_DEBUG: "Reactant,Reactant_jll"

0 commit comments

Comments
 (0)