Add type hints for reviewboard.scmtools.core.

Review Request #13064 — Created May 24, 2023 and submitted

Information

Review Board
release-6.x

Reviewers

This introduces type hints for some of our most important underlying
structures in Review Board. Types are now available for SCMTool,
SCMClient, Branch, Commit, ChangeSet, Revision,
FileLookupContext, and _SCMToolIDProperty.

This should help ensure proper usage of these classes, and proper
implementation of new SCMTools.

Some typing errors and and bad documentation were uncovered while
adding types. For instance, Popen had a bad string conversion and
bad documented return type, and some objects had unsafe comparison
logic. These have been fixed.

Unit tests pass.

Summary ID
Add type hints for reviewboard.scmtools.core.
This introduces type hints for some of our most important underlying structures in Review Board. Types are now available for `SCMTool`, `SCMClient`, `Branch`, `Commit`, `ChangeSet`, `Revision`, `FileLookupContext`, and `_SCMToolIDProperty`. This should help ensure proper usage of these classes, and proper implementation of new SCMTools.
f308f1e77fc131da5509f323d9814628a8c08685
david
  1. Ship It!
  2. 
      
chipx86
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-6.x (5dbdf5a)
Loading...