Added emojies, lines diff checker, app name by marinakalyuzhnaya · Pull Request #19 · ScaCap/code-coverage-assistant

Added emojies, lines diff checker, app name by marinakalyuzhnaya · Pull Request #19 · ScaCap/code-coverage-assistant

Problem solved:

added emojies to highlight comment with red/green colour for diff
added possibility to show details only for changed lines , please note, that this will work only if you have base lcov file, if not previous code will work without changes
added possibility to add app-name (needed mostly for singe repo with diff siblings)

Coverage after merging add-colors into master

100.00%

Coverage Report

File	Branches	Funcs	Lines	Uncovered Lines
src
html.js	100%	100%	100%
lcov.js	100%	100%	100%
tabulate.js	100%	100%	100%

Choose a reason for hiding this comment

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

Looks great, saw it in action with your other PR

Choose a reason for hiding this comment

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

Nice

marinakalyuzhnaya changed the title ~~Added emojies to the comment diff part~~ Added emojies, lines diff checker

marinakalyuzhnaya changed the title ~~Added emojies, lines diff checker~~ Added emojies, lines diff checker, app name

Choose a reason for hiding this comment

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

LGTM! 👍

Choose a reason for hiding this comment

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

Looks great again, I've seen it in action :)

Choose a reason for hiding this comment

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

Looks good. Few changes added.

Looks great again, I've seen it in action :)

I already have some improvements ideas, how to show only changed lines without having a base file. :)

Choose a reason for hiding this comment

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

Looks great. ❤️