Skip to content

YETUS-1276. Add --patch-mode flag to control diff-vs-patch preference#385

Merged
ndimiduk merged 1 commit into
apache:mainfrom
ndimiduk:YETUS-1276
May 21, 2026
Merged

YETUS-1276. Add --patch-mode flag to control diff-vs-patch preference#385
ndimiduk merged 1 commit into
apache:mainfrom
ndimiduk:YETUS-1276

Conversation

@ndimiduk
Copy link
Copy Markdown
Member

YETUS-983 hardcoded dryrun_both_files to prefer cumulative .diff over per-commit .patch. This may not work for all projects (binary handling is a known concern), so make the preference configurable.

--patch-mode=diff (default) preserves YETUS-983 behavior. --patch-mode=patch restores pre-YETUS-983 behavior. The non-preferred format is always tried as fallback.

@ndimiduk ndimiduk requested review from aw-was-here and cnauroth May 11, 2026 10:32
YETUS-983 hardcoded dryrun_both_files to prefer cumulative .diff over
per-commit .patch. This may not work for all projects (binary handling
is a known concern), so make the preference configurable.

--patch-mode=diff (default) preserves YETUS-983 behavior.
--patch-mode=patch restores pre-YETUS-983 behavior.
The non-preferred format is always tried as fallback.

fix shellcheck nit
@ndimiduk ndimiduk merged commit 8bff1d0 into apache:main May 21, 2026
3 checks passed
@ndimiduk ndimiduk deleted the YETUS-1276 branch May 21, 2026 09:19
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