flake8
passed.
JSHint
passed.
Review Request #12584 — Created Sept. 6, 2022 and submitted
We had a lot of inconsistency between
diff()
function signatures in
SCMClients. Some took only a subset of the arguments. Some took
arguments not defined inBaseSCMClient.diff()
. Git had its own
Git-specific argument,--git-find-renames-threshold
.This updates all function signatures to be compatible with the base
function signature.The Git-specific option has been removed. Instead,
GitClient.diff()
now just checksself.options
, instead ofrbt post
andrbt diff
doing this.
Unit tests pass.
Tested
rbt diff
andrbt post
with--git-find-renames-threshold
.
Summary | ID |
---|---|
453434e2516e851255b3814f1eadced3fb886776 |