Skip to content

Commit 3616155

Browse files
committed
update comments
1 parent 184ecca commit 3616155

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

.github/workflows/coverage.yml

+1
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ jobs:
3838
- name: Install coverage tool in in-project virtualenv
3939
run: |
4040
pdm run python -m ensurepip
41+
# coverage must run in the same venv as the code being tested.
4142
pdm run python -m pip install coverage
4243
- name: Generate coverage report
4344
run: |

.github/workflows/python-package.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ jobs:
4040
run: |
4141
echo "TARGET_PYTHON_VERSION_FOR_PDM=${{ matrix.python-version }}" | tr - @ >> "$GITHUB_ENV"
4242
# We need this hack at all because CI expects e.g. "pypy-3.10", whereas PDM expects "[email protected]".
43-
# Now that we have the result, send it to an environment variable so that the next step can use it.
43+
# We send the result into an environment variable so that the next step can use it.
4444
# https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/workflow-commands-for-github-actions#setting-an-environment-variable
4545
- name: Create in-project virtualenv and install dependencies
4646
run: |

0 commit comments

Comments
 (0)