Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Why does tox in Github Actions on Pull Request pull ALL the Pythons (**INCLUDING** 3.12)? #16

Closed
stdedos opened this issue Oct 18, 2023 · 2 comments
Closed

Why does tox in Github Actions on Pull Request pull ALL the Pythons (**INCLUDING** 3.12)? #16

stdedos opened this issue Oct 18, 2023 · 2 comments
Assignees
@stdedos
Labels
blocked bug Something isn't working CI upstream

Comments

@stdedos
Copy link
Collaborator

stdedos commented Oct 18, 2023

https://github.com/pylint-dev/pylint-pytest/actions/runs/6557075634/job/17831455987?pr=2

Tracked: https://support.github.com/ticket/personal/0/2385967
@stdedos stdedos added bug Something isn't working CI blocked upstream labels Oct 18, 2023
@stdedos stdedos self-assigned this Oct 18, 2023
@stdedos stdedos mentioned this issue Oct 18, 2023
Merged
@stdedos
Copy link
Collaborator Author

stdedos commented Oct 18, 2023

... ofc, the easy way would be to just not have a py312 in tox.ini, since we haven't fixed #3 ...

stdedos added a commit to stdedos/pylint-pytest that referenced this issue Oct 18, 2023
@stdedos
Fix testing + coverage
fa70388 
* Drop `py312` to combat pylint-dev#16
* Install editable, so that python uses the correct source files
* Ignore `tests/input/**/*.py` from coverage - they are not ran properly
* Again improve on artifact naming

Signed-off-by: Stavros Ntentos <[email protected]>
@stdedos stdedos mentioned this issue Oct 18, 2023
Merged
stdedos added a commit to stdedos/pylint-pytest that referenced this issue Oct 18, 2023
@stdedos
Fix testing + coverage
0a41a8e 
* Drop `py312` to combat pylint-dev#16
* Install editable, so that python uses the correct source files
* Ignore `tests/input/**/*.py` from coverage - they are not ran properly
* Again improve on artifact naming

Signed-off-by: Stavros Ntentos <[email protected]>
stdedos added a commit to stdedos/pylint-pytest that referenced this issue Oct 18, 2023
@stdedos
Fix testing + coverage
8406b11 
* Drop `py312` to combat pylint-dev#16
* Install `--editable`, so that python uses the correct source files
* Ignore `tests/input/**/*.py` from coverage - they are not ran properly
* Again improve on artifact naming

Signed-off-by: Stavros Ntentos <[email protected]>
stdedos added a commit to stdedos/pylint-pytest that referenced this issue Oct 18, 2023
@stdedos
Fix testing + coverage
c4a9409 
* Drop `py312` to combat pylint-dev#16
* Install `--editable`, so that Python uses the correct source files
* Ignore `tests/input/**/*.py` from coverage - they are not ran properly
* Again improve on artifact naming

Signed-off-by: Stavros Ntentos <[email protected]>
stdedos added a commit to stdedos/pylint-pytest that referenced this issue Oct 18, 2023
@stdedos
Fix testing + coverage
9f096e2 
* Drop `py312` to combat pylint-dev#16
* Install `--editable`, so that Python uses the correct source files
* Ignore `tests/input/**/*.py` from coverage - they are not ran properly
* Again improve on artifact naming

Signed-off-by: Stavros Ntentos <[email protected]>
stdedos added a commit to stdedos/pylint-pytest that referenced this issue Oct 18, 2023
@stdedos
Fix testing + coverage
fe28788 
* Drop `py312` to combat pylint-dev#16
* Install `--editable`, so that Python uses the correct source files
* Ignore `tests/input/**/*.py` from coverage - they are not ran properly
* Again improve on artifact naming

Signed-off-by: Stavros Ntentos <[email protected]>
stdedos added a commit to stdedos/pylint-pytest that referenced this issue Oct 18, 2023
@stdedos
Fix testing + coverage
c208abb 
* Drop `py312` to combat pylint-dev#16
* Install `--editable`, so that Python uses the correct source files for coverage
* Ignore `tests/input/**/*.py` from coverage - they are not ran properly
* Again improve on artifact naming

Signed-off-by: Stavros Ntentos <[email protected]>
stdedos added a commit that referenced this issue Dec 4, 2023
@stdedos
Debug #16 + https://support.github.com/ticket/personal/0/2385967
2f27ee8 
Signed-off-by: Stavros Ntentos <[email protected]>
@stdedos
Copy link
Collaborator Author

stdedos commented Dec 19, 2023

Internal GH stuff. We cannot do anything about it; issue comes from upstream.

Resolution: Just ignore it and run for all found Pythons; but ignore missing envs (c5af2e5).

@stdedos stdedos closed this as not planned Won't fix, can't repro, duplicate, stale Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stdedos stdedos

Labels
blocked bug Something isn't working CI upstream
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@stdedos