Update CVSClient for run_process(), type safety, and deprecations.
Review Request #12570 — Created Aug. 24, 2022 and submitted
Information | |
---|---|
chipx86 | |
RBTools | |
release-4.x | |
12571 | |
Reviewers | |
rbtools | |
CVSClient
now usesrun_process()
instead ofexecute()
for all
cvs
interaction, giving us better type safety and safer execution.All methods have been updated with type annotations, ensuring compliance
withBaseSCMClient
requirements. Documentation has been updated where
appropriate.Some deprecated calls have been updated, and dead code removed.
Unit tests pass on Python 3.7-3.11.
Change Summary:
- Removed unused keyword arguments and moved them into
**kwargs
.
Commits: |
|
|||||||||
---|---|---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+154 -132) |
Checks run (2 succeeded)
flake8
passed.
JSHint
passed.