Skip to content

v.util: fix diff coloring, add test - #21260

Merged
spytheman merged 6 commits into
vlang:masterfrom
ttytm:util/fix-diff-coloring
Apr 13, 2024
Merged

spytheman merged 6 commits into
vlang:masterfrom
ttytm:util/fix-diff-coloring

Conversation

@ttytm

@ttytm ttytm commented Apr 12, 2024 •

Copy link
Copy Markdown
Member

The PR should fix coloring when using a system default diff comparison and adds a test if the output actually includes color codes.

@ttytm
ttytm force-pushed the util/fix-diff-coloring branch from edd33b7 to 88501b4 Compare April 12, 2024 00:09
@ttytm
ttytm force-pushed the util/fix-diff-coloring branch from 88501b4 to 173cd8d Compare April 12, 2024 00:24
Comment thread vlib/v/util/diff/diff_test.v Outdated
Comment thread vlib/v/util/diff/diff.v
@spytheman
spytheman merged commit b838418 into vlang:master Apr 13, 2024
@ttytm
ttytm deleted the util/fix-diff-coloring branch April 13, 2024 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants