Summary

Ensure that all SCMClient.diff methods take kwargs.

Review Request #11581 — Created April 7, 2021 and submitted May 13, 2021, 10:49 a.m.

Information

Owner

david

Repository

RBTools

Branch

master

Bugs

Depends On

Reviewers

Groups

rbtools

People

Description

The vast majority of the definitions of the diff method for various

SCMClient subclasses included kwargs, for future expansion. There were

two places, the base class definition, and the TFS subclass, which

didn't. This change adds those so that I can later pass the

RepositoryInfo object into this method safely.

Testing Done

Ran unit tests.

Commits

Summary	ID
Ensure that all SCMClient.diff methods take kwargs. The vast majority of the definitions of the diff method for various SCMClient subclasses included kwargs, for future expansion. There were two places, the base class definition, and the TFS subclass, which didn't. This change adds those so that I can later pass the RepositoryInfo object into this method safely. Testing Done: Ran unit tests.	33a7d44ba20070b457287025edb66617f96e5f2f

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to master (eef797d)