Skip to content

Commit 6186781

Browse files
Apply suggestions from code review
Co-authored-by: George L. Yermulnik <[email protected]>
1 parent dea6ba1 commit 6186781

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

.github/workflows/reusable-tox.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -274,7 +274,7 @@ jobs:
274274
path: dist/
275275

276276
- name: >-
277-
Pre-populate tox envs: ${{ env.TOXENV }}
277+
Pre-populate tox envs: `${{ env.TOXENV }}`
278278
shell: bash
279279
env:
280280
INPUT_BUILT_WHEEL_NAMES: ${{ inputs.built-wheel-names }}
@@ -284,7 +284,7 @@ jobs:
284284
--parallel-live
285285
--skip-missing-interpreters false
286286
)
287-
if [ -n "$INPUT_BUILT_WHEEL_NAMES" ]; then
287+
if [[ $INPUT_BUILT_WHEEL_NAMES ]]; then
288288
python -Im tox "${tox_common_args[@]}" \
289289
--installpkg "dist/$INPUT_BUILT_WHEEL_NAMES" \
290290
--notest
@@ -307,7 +307,7 @@ jobs:
307307
# But only for 'pytest' env in 'tox'.
308308
# For details: ../../tox.ini '[testenv:pytest]' 'commands_post'
309309
- name: >-
310-
Run tox envs: ${{ env.TOXENV }}
310+
Run tox envs: `${{ env.TOXENV }}`
311311
id: tox-run
312312
shell: bash
313313
env:
@@ -365,7 +365,7 @@ jobs:
365365
&& steps.tox-run.outputs.test-result-files == ''
366366
&& steps.tox-run.outputs.codecov-flags != 'MyPy'
367367
run: >-
368-
cat code-coverage-results.md >> $GITHUB_STEP_SUMMARY
368+
cat code-coverage-results.md >> "$GITHUB_STEP_SUMMARY"
369369
- name: Re-run the failing tests with maximum verbosity
370370
if: >-
371371
!cancelled()

0 commit comments

Comments
 (0)