Skip to content

Conversation

@hategan
Copy link
Collaborator

@hategan hategan commented Mar 20, 2023

…tests

are set up from.

Also, make sure that dependencies are cleaned between branch tests since there are cases when pip doesn't downgrade correctly, which can lead to tests failing when they shouldn't.

…tests

are set up from.

Also, make sure that dependencies are cleaned between branch tests since
there are cases when pip doesn't downgrade correctly, which can lead to
tests failing when they shouldn't.
@hategan hategan requested review from andre-merzky and mtitov March 20, 2023 22:09
@codecov
Copy link

codecov bot commented Mar 20, 2023

Codecov Report

Merging #366 (9e203f1) into main (a98c04e) will decrease coverage by 0.12%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##             main     #366      +/-   ##
==========================================
- Coverage   69.84%   69.72%   -0.12%     
==========================================
  Files          74       74              
  Lines        3164     3168       +4     
==========================================
- Hits         2210     2209       -1     
- Misses        954      959       +5     
Impacted Files Coverage Δ
tests/ci_runner.py 20.44% <16.66%> (+0.10%) ⬆️

... and 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Collaborator

@andre-merzky andre-merzky left a comment

Choose a reason for hiding this comment

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

Makes sense to me.

@hategan hategan merged commit 2a4a470 into main Mar 21, 2023
@hategan hategan deleted the clean_pip_install_dirs branch March 21, 2023 21:05
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.

3 participants