search

mkaz / termgraph

a python command-line tool which draws basic graphs in the terminal
MIT License
3.15k stars 164 forks source link

Fix vertical graphs and don't reset colors if not in use #32

Closed joaquincasares closed 6 years ago

joaquincasares commented 6 years ago

Hey again!

Nice work since the last time I used the project! :)

I spotted two bugs and wanted to patch them.

The first is an error when using --vertical. It was expecting args to be present, but now I'm passing it into vertically().

The second is one that was messing up my piped output to a text file since invisible resets were showing up in my generated report.md. The code now only resets colors if it was using colors to begin with.

Thanks again for the great project!

mkaz commented 6 years ago

Thanks @joaquincasares for the PR!