deis / controller

Deis Workflow Controller (API)
https://deis.com
MIT License
41 stars 53 forks source link

chore(requirements): update requests to 2.13.0 #1235

Closed mboersma closed 7 years ago

mboersma commented 7 years ago

From the requests library release notes:

Features

Miscellaneous

codecov-io commented 7 years ago

Codecov Report

Merging #1235 into master will not impact coverage.

@@           Coverage Diff           @@
##           master    #1235   +/-   ##
=======================================
  Coverage   87.29%   87.29%           
=======================================
  Files          43       43           
  Lines        3826     3826           
  Branches      665      665           
=======================================
  Hits         3340     3340           
  Misses        321      321           
  Partials      165      165

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 98fd5bc...99c11d9. Read the comment docs.

mboersma commented 7 years ago

This actually breaks a regex-based unit test, so I'll work that out and then resubmit it for review.

mboersma commented 7 years ago

My rewriting of import statements somehow had caused the unit test failure, so I reverted that. Works fine now: ready for review again.