search

kevin1024 / vcrpy

Automatically mock your HTTP interactions to simplify and speed up testing
MIT License
2.72k stars 388 forks source link

Remove unnecessary dependency on six. #743

Closed charettes closed 1 year ago

charettes commented 1 year ago

Remove the last remaining usage of it in VCR.testcase.

codecov-commenter commented 1 year ago

Codecov Report

Merging #743 (3ff822d) into master (7e11cfc) will decrease coverage by 0.12%. The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##           master     #743      +/-   ##
==========================================
- Coverage   90.63%   90.52%   -0.12%     
==========================================
  Files          28       28              
  Lines        1794     1794              
  Branches      322      322              
==========================================
- Hits         1626     1624       -2     
- Misses        133      135       +2     
  Partials       35       35
Impacted Files Coverage Δ
vcr/config.py 95.06% <100.00%> (ø)

... and 1 file with indirect coverage changes

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

charettes commented 1 year ago

@hartwork sure thing, thanks for the review! Just forced-pushed a commit that addresses the formatting issue.