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.
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
Only test with coverage on pushes to sphinx-doc/sphinx
#10361
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. But it seems CI are not passed. Could you check them please?
@@ -47,8 +47,7 @@ | |||
"types-requests", | |||
], | |||
'test': [ | |||
'pytest', | |||
'pytest-cov', | |||
'pytest>=4.6', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note: pytest-4.6 has been packaged since Ubuntu 20.04. But I think it's okay because this dependency is for developers, not users.
Ubuntu LTS | pytest version |
---|---|
18.04 | 3.3.2 |
20.04 | 4.6.9 |
22.04 | 6.2.5 |
172061a
to
65ae948
Compare
Updated, I forgot that generators aren't reversible. Seems to be passing now. A |
Thank you for quick update. Merging now. |
@AA-Turner The coverage job was failed just after merging this. Could you check this, please? |
Was just working on that, please see #10413 -- sorry for the oversight. A |
To speed up CI for PRs, this moves the coverage tests to only run on pushes to branches in this repo, rather than on forks and pull requests too.
I also bumped
pytest
to 4.6 and removed some now-redundant code (we had previously always installedpytest-cov
, which has requiredpytest>=4.6
since 2020)https://github.com/pytest-dev/pytest-cov/blob/694f7fd497b07aaea76d96e484068faf440e8301/setup.py#L125
A
Feature or Bugfix
Relates