Add a new extension hook for registering SCMTools.

Review Request #12395 — Created June 21, 2022 and submitted

Information

Review Board
release-5.0.x

Reviewers

Now that SCMTools are backed by a registry instead of relying entirely
on entry points and the registerscmtools management command, we can
make it easier for extensions to add new ones. This change adds a new
extension hook, SCMToolHook, for doing so. It works very similarly to
the existing hook for hosting services.

  • Ran unit tests.
  • Built and reviewed docs.
Summary ID
Add a new extension hook for registering SCMTools.
Now that SCMTools are backed by a registry instead of relying entirely on entry points and the `registerscmtools` management command, we can make it easier for extensions to add new ones. This change adds a new extension hook, `SCMToolHook`, for doing so. It works very similarly to the existing hook for hosting services. Testing Done: - Ran unit tests. - Built and reviewed docs.
627f310e757b2726b6cc57c29c3caa05032a14cf
Description From Last Updated

"used for configured repositories" or "used to configure repositories"?

chipx86chipx86

Too many "

chipx86chipx86

Let's use assertIs here, for completeness.

chipx86chipx86
chipx86
  1. Neat.

  2. "used for configured repositories" or "used to configure repositories"?

  3. reviewboard/extensions/hooks.py (Diff revision 1)
     
     

    Too many "

  4. Let's use assertIs here, for completeness.

  5. 
      
david
chipx86
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-5.0.x (3fece69)
Loading...