Modernize hosting service registry management.

Review Request #13181 — Created Aug. 2, 2023 and submitted — Latest diff uploaded

Information

Review Board
release-6.x

Reviewers

This makes a handful of changes for the hosting service registry, adding
typing, new documentation, and utility methods designed to replace those
in reviewboard.hostingsvcs.service.

HostingServiceRegistry now has convenience methods for retrieving or
unregistering a hosting service by ID. Logging from the methods in
service.py have also been added. With this, the service.py code is
now just a thin wrapper over the registry.

Pre-population of the registry via the initializing signal now happens
within reviewboard/hostingsvcs/__init__.py, which is guaranteed to be
run. It no longer relies on service.py being imported somewhere.

All unit tests pass.

Diff Revision 1 (Latest)

Commits

First Last Summary ID Author
Modernize hosting service registry management.
This makes a handful of changes for the hosting service registry, adding typing, new documentation, and utility methods designed to replace those in `reviewboard.hostingsvcs.service`. `HostingServiceRegistry` now has convenience methods for retrieving or unregistering a hosting service by ID. Logging from the methods in `service.py` have also been added. With this, the `service.py` code is now just a thin wrapper over the registry. Pre-population of the registry via the `initializing` signal now happens within `reviewboard/hostingsvcs/__init__.py`, which is guaranteed to be run. It no longer relies on `service.py` being imported somewhere.
88dfddc3c7e505d693fc158c4f02ba0df3d5941f Christian Hammond
reviewboard/hostingsvcs/__init__.py
reviewboard/hostingsvcs/service.py
reviewboard/hostingsvcs/base/registry.py
Loading...