Update diff functions on all SCMClients and remove client-specific args.
Review Request #12584 — Created Sept. 6, 2022 and submitted — Latest diff uploaded
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 in
BaseSCMClient.diff(). Git had its own
This updates all function signatures to be compatible with the base
The Git-specific option has been removed. Instead,
now just checks
self.options, instead of
Unit tests pass.