Codecov Report

Merging #63 into master will increase coverage by 0.06%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master      #63      +/-   ##
============================================
+ Coverage      88.7%   88.76%   +0.06%     
- Complexity      237      239       +2     
============================================
  Files            18       18              
  Lines           726      730       +4     
============================================
+ Hits            644      648       +4     
  Misses           82       82

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 9e7c884...1eee3d6. Read the comment docs.

hey @robertboloc thanks for the contribution 😄 ... looking back at this I personally think it might be best that we don't throw an exception as this might be unwanted behaviour, e.g. if I build an app but I don't expect someone to be using it with a terminal width less than 20 but someone does and they get an exception that doesn't quite make sense to them as a user. What do you think ?

I'm personally leaning more towards silently not adding the ascii art or replacing it with a small ascii art sign (something to represent it being an image but a failed one just for amusement really) ... but both would represent keeping functionality for the end user, just sacrificing aesthetics in most cases.

@AydinHassan have you got any thoughts on this ?

hi @mikeymike that makes sense, I added the exception thinking of it from a development point of view, but you are right that for the end user it's a bad experience. I will skip adding the ascii art if it fails the check. That will still allow using the app without the art.