Skip to content

Conversation

tleonhardt
Copy link
Member

Removed a Python 2/3 compatibility hack which is no longer necessary due to recent modifications in how the load command works.

Also added some comments and a unit test of how the command echo functionality works when a script is running.

@tleonhardt tleonhardt added this to the 0.7.5 milestone Jul 8, 2017
@tleonhardt tleonhardt self-assigned this Jul 8, 2017
@codecov
Copy link

codecov bot commented Jul 8, 2017

Codecov Report

Merging #178 into master will increase coverage by 0.07%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #178      +/-   ##
==========================================
+ Coverage   96.37%   96.45%   +0.07%     
==========================================
  Files           1        1              
  Lines        1130     1127       -3     
==========================================
- Hits         1089     1087       -2     
+ Misses         41       40       -1
Impacted Files Coverage Δ
cmd2.py 96.45% <100%> (+0.07%) ⬆️

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 22c43d7...af248f9. Read the comment docs.

@tleonhardt tleonhardt merged commit a140ad1 into master Jul 8, 2017
@tleonhardt tleonhardt deleted the remove_filehack branch July 8, 2017 16:34
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.

1 participant