greglook / puget

Canonical Colorizing Clojure Printer
The Unlicense
252 stars 27 forks source link

Ansi codes now respect indentation #46

Closed lread closed 5 years ago

lread commented 5 years ago

Bumped fipp to 0.6.21 to pick up https://github.com/brandonbloom/fipp/issues/66 which resolves #44.

To verify, beefed up ansi unit tests to include a test that almost matches example in puget README and a test that clearly shows indentation.

Of note: While developing these tests I used kaocha because it has excellent expected vs actual diff reporting. Because kaocha uses puget, I figured it might not be the best idea to use kaocha for puget testing and therefore have not proposed to include it.

codecov-io commented 5 years ago

Codecov Report

Merging #46 into develop will increase coverage by 0.23%. The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #46      +/-   ##
===========================================
+ Coverage     99.3%   99.53%   +0.23%     
===========================================
  Files            5        5              
  Lines          431      431              
  Branches         1        1              
===========================================
+ Hits           428      429       +1     
+ Misses           2        1       -1     
  Partials         1        1
Impacted Files Coverage Δ
src/puget/color/ansi.clj 100% <0%> (+4.76%) :arrow_up:

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 d438931...d9ed9b8. Read the comment docs.