Skip to content

Commit f819b86

Browse files
CI: don't unset default interpreter when setting pex tool (#268)
The `GITHUB_ENV` file doesn't persist between steps - changes made to the environment via that file are reflected in the environments of all future steps. Concatenate extra options onto `PLZ_ARGS` by reading its current value from the environment and writing its desired future value to the `GITHUB_ENV` file.
1 parent 4d063d1 commit f819b86

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/workflows/plugin_test.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ jobs:
4141
- name: Configure plugin to use please_pex tool built from source
4242
if: inputs.please_pex_from_repo
4343
run: |
44-
echo "PLZ_ARGS="$(grep ^PLZ_ARGS= $GITHUB_ENV | cut -d= -f2-)" -o plugin.python.pextool://tools/please_pex" >> $GITHUB_ENV
44+
echo "PLZ_ARGS=${PLZ_ARGS:+$PLZ_ARGS }-o plugin.python.pextool://tools/please_pex" >> $GITHUB_ENV
4545
- name: Run tests
4646
run: ./pleasew test --keep_going --log_file plz-out/log/test.log ${{ inputs.test_targets }}
4747
- name: Archive logs

0 commit comments

Comments
 (0)