Skip to content

[enhancement] Add flag '--ignore-blank-lines' when ignoring whitespace#2455

Open
goran-w wants to merge 1 commit into
sourcegit-scm:developfrom
goran-w:ignore-blank-lines-along-with-whitespace
Open

[enhancement] Add flag '--ignore-blank-lines' when ignoring whitespace#2455
goran-w wants to merge 1 commit into
sourcegit-scm:developfrom
goran-w:ignore-blank-lines-along-with-whitespace

Conversation

@goran-w

@goran-w goran-w commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Inspired by a discussion in PR #2411.

  • With the added option --ignore-blank-lines, some more cases of changes in (added / removed) empty lines will be ignored (but apparently not when there are also other unignored changes in the same file).
  • Still, it's a useful addition wherever it applies, since it helps in quickly skipping over files that have only empty-line / line-break changes (by displaying the No changes or only EOL changes panel).
  • Also renamed argument unified to numContextLines, for clarity.

…ace changes

* This causes some more cases of changes in (added/removed) empty lines to be ignored (but apparently not when there also other unignored changes in the same file).
* Still, it's a useful addition wherever it applies, since it helps in quickly skipping over files that have only empty-line changes.
* Related to discussion in PR sourcegit-scm#2411.
* Also renamed argument 'unified' to 'numContextLines', for clarity.
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.

1 participant