ansi-to-html icon indicating copy to clipboard operation
ansi-to-html copied to clipboard

Main coverage

Open brettz9 opened this issue 6 years ago • 5 comments

With this PR, there is now 100% test coverage.

You should take an especially good look at the last 4 commits (before the last commit which is just an npm update), however, as I'm not sure if that apparently unreachable code I've removed had some other valid intention behind it.

brettz9 avatar Apr 13 '20 05:04 brettz9

Thanks for the review/merges! I've rebased this now as well as rebased #70.

brettz9 avatar Apr 22 '20 15:04 brettz9

thanks for all the pull requests. I'm unsure of the deleted code in this one. I haven't had a chance to look closely at it. But, will try to do that soon.

rburns avatar Apr 23 '20 00:04 rburns

Just a friendly reminder if you're not too busy...

brettz9 avatar May 12 '20 00:05 brettz9

If you are ok with occasional reminders, here's a bump...

brettz9 avatar Jul 23 '20 00:07 brettz9

thanks for the reminder. I hope to get back to taking a look at this soon.

rburns avatar Aug 26 '20 06:08 rburns