Skip to content

Commit

Permalink
chore: Rename 'py' to 'pytest' in GHA to avoid confusion (#749)
Browse files Browse the repository at this point in the history
When you see `py` in tests it not clear what exactly going on. Until now I thought that it represents "python", but actually this part run `pytest`
![image](https://github.com/user-attachments/assets/a7f5aeef-a4ab-4d18-a793-834f0b3c5ce9)

So, for avoiding confusion and add ability to explicitly specify `tox r -e pytest`, this PR was made
  • Loading branch information
MaxymVlasov authored Jan 13, 2025
1 parent 5181957 commit e251eb2
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci-cd.yml
Original file line number Diff line number Diff line change
Expand Up @@ -452,7 +452,7 @@ jobs:
- macos-13
- windows-2025
toxenv:
- py
- pytest
xfail:
- false

Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/reusable-tox.yml
Original file line number Diff line number Diff line change
Expand Up @@ -262,7 +262,7 @@ jobs:
shell: bash
- name: Download all the dists
if: >-
contains(fromJSON('["metadata-validation", "py"]'), inputs.toxenv)
contains(fromJSON('["metadata-validation", "pytest"]'), inputs.toxenv)
uses: actions/download-artifact@v4
with:
name: ${{ inputs.dists-artifact-name }}
Expand Down
3 changes: 3 additions & 0 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,9 @@ set_env =
COVERAGE_PROCESS_START = {toxinidir}{/}.coveragerc
wheel_build_env = .pkg

# Duplicate default 'py' env to 'pytest' to be able run pytest with 'tox run -e pytest'
[testenv:pytest]


[testenv:cleanup-dists]
description =
Expand Down

0 comments on commit e251eb2

Please sign in to comment.