Summary

Add formal deprecation notices for reviewboard.hostingsvcs.service.

Review Request #14584 — Created Sept. 4, 2025 and submitted Sept. 15, 2025, 9:31 a.m.

Information

Owner

david

Repository

Review Board

Branch

release-7.1.x

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

A while ago, we moved most of the actual base hosting service
implementation into submodules of reviewboard.hostingsvcs.base. The
service module had forwarding imports and a comment about deprecation,
but we didn't have any actual deprecation warnings. This adds
deprecation warnings for everything in that module, and updates
dependent code to use the new names.

Testing Done

Ran unit tests.

Commits

Summary	ID
Add formal deprecation notices for reviewboard.hostingsvcs.service. A while ago, we moved most of the actual base hosting service implementation into submodules of `reviewboard.hostingsvcs.base`. The `service` module had forwarding imports and a comment about deprecation, but we didn't have any actual deprecation warnings. This adds deprecation warnings for everything in that module, and updates dependent code to use the new names. Testing Done: - Ran unit tests.	rpyzqmxqovktnxxlxzzrspnsurtqyxyk

Summary

Add formal deprecation notices for reviewboard.hostingsvcs.service.

A while ago, we moved most of the actual base hosting service implementation into submodules of `reviewboard.hostingsvcs.base`. The `service` module had forwarding imports and a comment about deprecation, but we didn't have any actual deprecation warnings. This adds deprecation warnings for everything in that module, and updates dependent code to use the new names. Testing Done: - Ran unit tests.

rpyzqmxqovktnxxlxzzrspnsurtqyxyk

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Change Summary:

Move to release-7.1.x

Commits:

	Summary	ID
	Add formal deprecation notices for reviewboard.hostingsvcs.service. A while ago, we moved most of the actual base hosting service implementation into submodules of `reviewboard.hostingsvcs.base`. The `service` module had forwarding imports and a comment about deprecation, but we didn't have any actual deprecation warnings. This adds deprecation warnings for everything in that module, and updates dependent code to use the new names. Testing Done: - Ran unit tests.	ytvwtzvwntrqnoonzrntxlsxokzqtuuv
	Add formal deprecation notices for reviewboard.hostingsvcs.service. A while ago, we moved most of the actual base hosting service implementation into submodules of `reviewboard.hostingsvcs.base`. The `service` module had forwarding imports and a comment about deprecation, but we didn't have any actual deprecation warnings. This adds deprecation warnings for everything in that module, and updates dependent code to use the new names. Testing Done: - Ran unit tests.	rpyzqmxqovktnxxlxzzrspnsurtqyxyk

Branch:

master

release-7.1.x

Diff:

Revision 2 (+272 -156)

Show changes

	reviewboard/hostingsvcs/assembla.py
	reviewboard/hostingsvcs/beanstalk.py
	reviewboard/hostingsvcs/bitbucket.py
	reviewboard/hostingsvcs/bugzilla.py
	reviewboard/hostingsvcs/codebasehq.py
	reviewboard/hostingsvcs/fake.py
	reviewboard/hostingsvcs/fedorahosted.py
	reviewboard/hostingsvcs/fogbugz.py
	19 more

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Change Summary:

Fix version comments.

Commits:

	Summary	ID
	Add formal deprecation notices for reviewboard.hostingsvcs.service. A while ago, we moved most of the actual base hosting service implementation into submodules of `reviewboard.hostingsvcs.base`. The `service` module had forwarding imports and a comment about deprecation, but we didn't have any actual deprecation warnings. This adds deprecation warnings for everything in that module, and updates dependent code to use the new names. Testing Done: - Ran unit tests.	rpyzqmxqovktnxxlxzzrspnsurtqyxyk
	Add formal deprecation notices for reviewboard.hostingsvcs.service. A while ago, we moved most of the actual base hosting service implementation into submodules of `reviewboard.hostingsvcs.base`. The `service` module had forwarding imports and a comment about deprecation, but we didn't have any actual deprecation warnings. This adds deprecation warnings for everything in that module, and updates dependent code to use the new names. Testing Done: - Ran unit tests.	rpyzqmxqovktnxxlxzzrspnsurtqyxyk

Diff:

Revision 3 (+272 -156)

Show changes

	reviewboard/hostingsvcs/assembla.py
	reviewboard/hostingsvcs/beanstalk.py
	reviewboard/hostingsvcs/bitbucket.py
	reviewboard/hostingsvcs/bugzilla.py
	reviewboard/hostingsvcs/codebasehq.py
	reviewboard/hostingsvcs/fake.py
	reviewboard/hostingsvcs/fedorahosted.py
	reviewboard/hostingsvcs/fogbugz.py
	19 more

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to release-7.1.x (7b84dee)