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

Make small tox/pytest config improvements #430

Merged
merged 5 commits into from
Oct 30, 2023
Merged

Make small tox/pytest config improvements #430

merged 5 commits into from
Oct 30, 2023

Conversation

sevein
Copy link
Member

@sevein sevein commented Oct 30, 2023

No description provided.

A previous commit introduced pytest flags `-Wd` inadvertently, causing
all warnings to be printed but not causing tests to fails. This commit
removes such flags when invoking pytest since we rely on `filterwarnigs`
to ignore specific deprecation warnings (mostly `pkg_resources`
related).
@codecov-commenter
Copy link

codecov-commenter commented Oct 30, 2023

Codecov Report

Merging #430 (c7e26e1) into main (f922286) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #430   +/-   ##
=======================================
  Coverage   45.89%   45.89%           
=======================================
  Files         104      104           
  Lines        7384     7384           
  Branches     1200     1200           
=======================================
  Hits         3389     3389           
  Misses       3824     3824           
  Partials      171      171           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@sevein sevein merged commit 238bed1 into main Oct 30, 2023
5 checks passed
@sevein sevein deleted the dev/small-improvs branch October 30, 2023 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants