Skip to content

Allow omitting whitespace only changes #222

New issue
New issue
Closed
#385
Closed
Allow omitting whitespace only changes#222
#385
Assignees
HeavyWombat
Labels
enhancementNew feature or request
@clux

Description

@clux
clux
opened on May 5, 2022

Hey all, thanks a lot for maintaining this great tool!

This is a feature request/suggestion for isWhitespaceOnlyChanges:

dyff/pkg/dyff/output_human.go

Lines 337 to 342 in 416d58b

case isWhitespaceOnlyChange(from, to):
_, _ = output.WriteString(yellow("%c whitespace only change\n", MODIFICATION))
report.writeTextBlocks(output, 0,
red("%s", createStringWithPrefix(" - ", showWhitespaceCharacters(from))),
green("%s", createStringWithPrefix(" + ", showWhitespaceCharacters(to))),
)

It would be amazing to have a --omit-whitespace flag on dyff between.
WDYT?

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions