krallin commented 7 years ago

To pull in this fix: https://github.com/erikhuda/thor/pull/575

cc @UserNotFound - this fixes the issue you ran into here: https://aptible.slack.com/archives/C0GN45N59/p1508457537000162 cc @fancyremarker

krallin commented 7 years ago

In the way of verifying this change, I am:

Running the integration tests that invoke the CLI
Going through the Thor changelog.
And... testing out a few things manually

codecov-io commented 7 years ago

Codecov Report

Merging #239 into master will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #239   +/-   ##
=======================================
  Coverage   95.67%   95.67%           
=======================================
  Files          58       58           
  Lines        2772     2772           
=======================================
  Hits         2652     2652           
  Misses        120      120

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 bf681a8...b6569cb. Read the comment docs.

aptible / aptible-cli

Bump Thor to 0.20.0 #239

Codecov Report