From e251eb2e2e23cf4b1c35fcce789ff70ecc3f55b3 Mon Sep 17 00:00:00 2001 From: Maksym Vlasov Date: Mon, 13 Jan 2025 22:56:25 +0200 Subject: [PATCH] chore: Rename 'py' to 'pytest' in GHA to avoid confusion (#749) 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 --- .github/workflows/ci-cd.yml | 2 +- .github/workflows/reusable-tox.yml | 2 +- tox.ini | 3 +++ 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/.github/workflows/ci-cd.yml b/.github/workflows/ci-cd.yml index 0e18ee217..4f05785af 100644 --- a/.github/workflows/ci-cd.yml +++ b/.github/workflows/ci-cd.yml @@ -452,7 +452,7 @@ jobs: - macos-13 - windows-2025 toxenv: - - py + - pytest xfail: - false diff --git a/.github/workflows/reusable-tox.yml b/.github/workflows/reusable-tox.yml index b540075e4..bcd1a2182 100644 --- a/.github/workflows/reusable-tox.yml +++ b/.github/workflows/reusable-tox.yml @@ -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 }} diff --git a/tox.ini b/tox.ini index 195db1536..adb9defac 100644 --- a/tox.ini +++ b/tox.ini @@ -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 =