Skip to content

Conversation

@kmvanbrunt
Copy link
Member

@kmvanbrunt kmvanbrunt commented Dec 3, 2019

Closes #824
Added self to perror() for consistency and ease of overriding
poutput(), perror(), and pwarning() can now be called with no args

poutput(), perror(), and pwarning() can now be called with no args
@codecov
Copy link

codecov bot commented Dec 3, 2019

Codecov Report

Merging #827 into master will decrease coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #827      +/-   ##
==========================================
- Coverage   97.23%   97.23%   -0.01%     
==========================================
  Files          14       14              
  Lines        3513     3512       -1     
==========================================
- Hits         3416     3415       -1     
  Misses         97       97
Impacted Files Coverage Δ
cmd2/cmd2.py 96.49% <100%> (-0.01%) ⬇️

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 7722be4...9e8d85a. Read the comment docs.

Copy link
Member

@tleonhardt tleonhardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@tleonhardt tleonhardt merged commit a112f14 into master Dec 3, 2019
@tleonhardt tleonhardt deleted the printer_changes branch December 3, 2019 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Why is perror() a static method?

3 participants