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

Fix test failure #520

Merged
merged 1 commit into from
May 5, 2023
Merged

Fix test failure #520

merged 1 commit into from
May 5, 2023

Conversation

agateau-gg
Copy link
Collaborator

vcrpy does not use urllib3 correctly. Pin urllib3 until kevin1024/vcrpy#688 is fixed.

@codecov-commenter
Copy link

codecov-commenter commented May 4, 2023

Codecov Report

Merging #520 (2575101) into main (de0a63f) will increase coverage by 0.05%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main     #520      +/-   ##
==========================================
+ Coverage   93.81%   93.87%   +0.05%     
==========================================
  Files          80       83       +3     
  Lines        3818     3839      +21     
==========================================
+ Hits         3582     3604      +22     
+ Misses        236      235       -1     
Flag Coverage Δ
unittests 93.87% <ø> (+0.05%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 23 files with indirect coverage changes

@agateau-gg agateau-gg force-pushed the agateau/unbreak-tests branch 7 times, most recently from 2575101 to c652798 Compare May 5, 2023 11:53
vcrpy does not use urllib3 correctly. Pin urllib3 until
kevin1024/vcrpy#688 is fixed.

Needed a hack in the "Install dependencies" step to work around a build
failure on Windows: for some reason the Windows builder still had urllib
2.0.2 installed. Could not reproduce this in my Windows VM so I suspect
there is a bug in the way the setup-python GitHub action works.
@agateau-gg agateau-gg force-pushed the agateau/unbreak-tests branch from c652798 to e641df3 Compare May 5, 2023 12:55
@agateau-gg agateau-gg merged commit 05ccbf7 into main May 5, 2023
@agateau-gg agateau-gg deleted the agateau/unbreak-tests branch May 5, 2023 13:07
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.

2 participants