search

BBVA / chaos-monkey-engine

A Chaos Engineering swiss army knife
Apache License 2.0
45 stars 6 forks source link

Fixes to improve python2.7 compatibility #12

Closed alexkas1977 closed 6 years ago

gdiazlo commented 6 years ago

Thanks for your request and contributions. In order to accept the change, we need it to build successfully. There are spacing issues in the pull request, can you fix them and see if travis build finished correctly? thanks

codecov-io commented 6 years ago

Codecov Report

Merging #12 into master will not change coverage. The diff coverage is 80%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #12   +/-   ##
=======================================
  Coverage   80.57%   80.57%           
=======================================
  Files          25       25           
  Lines         767      767           
=======================================
  Hits          618      618           
  Misses        149      149
Impacted Files Coverage Δ
chaosmonkey/attacks/executor.py 36.36% <0%> (ø) :arrow_up:
chaosmonkey/engine/cme_manager.py 93.15% <100%> (ø) :arrow_up:
chaosmonkey/attacks/attack.py 83.33% <100%> (ø) :arrow_up:
chaosmonkey/planners/planner.py 72.22% <100%> (ø) :arrow_up:

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 f4f5fbe...b073375. Read the comment docs.