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.

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

chore: update pytest-xdist package #290

Merged
merged 3 commits into from Sep 26, 2023
Merged

Conversation

RomanBredehoft
Copy link
Collaborator

This removes the py dependency, which has a high risk vulnerability : https://github.com/zama-ai/concrete-ml/security/dependabot/8

@RomanBredehoft RomanBredehoft requested a review from a team as a code owner September 25, 2023 16:34
@cla-bot cla-bot bot added the cla-signed label Sep 25, 2023
@RomanBredehoft RomanBredehoft changed the title chore: update ptest-xdist package chore: update pytest-xdist package Sep 25, 2023
fd0r
fd0r previously approved these changes Sep 26, 2023
Copy link
Collaborator

@fd0r fd0r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great! Thanks for that

@@ -135,6 +135,7 @@ filterwarnings = [
"ignore:Implicitly cleaning up:ResourceWarning",
"ignore:non-integer arguments to randrange\\(\\) have been deprecated since Python 3\\.10 and will be removed in a subsequent version:DeprecationWarning",
"ignore:Deprecated call to `pkg_resources.declare_namespace:DeprecationWarning",
"ignore:The --rsyncdir command line argument and rsyncdirs config variable are deprecated.:DeprecationWarning",
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a non-breaking compatibility problem between pytest-cov and pytest-xdist 3.x : pytest-dev/pytest-cov#557 (closed but not fixed) , so we need to filter out this warning

@github-actions
Copy link

Coverage passed ✅

Coverage details

---------- coverage: platform linux, python 3.8.18-final-0 -----------
Name    Stmts   Miss  Cover   Missing
-------------------------------------
TOTAL    6164      0   100%

51 files skipped due to complete coverage.

@RomanBredehoft RomanBredehoft merged commit a248bde into main Sep 26, 2023
9 checks passed
@RomanBredehoft RomanBredehoft deleted the chore/upgrade_pytest_xdist branch September 26, 2023 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants