Summary

Update typing: Union[...]

Review Request #15119 — Created June 12, 2026 and updated June 12, 2026, 11:08 a.m.

Information

Owner

david

Repository

Review Board

Branch

release-9.x

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

This change updates legacy type hints that use the old Union[...] form
to be (... | ...) instead.

Because I want to keep this tightly focused and easy to review, this
does not make any changes to other imports, documentation, etc., even
within the changed lines. Those will come in future changes.

Testing Done

Ran unit tests.

Commits

Summary	ID
Update typing: Union[...] This change updates legacy type hints that use the old `Union[...]` form to be `(... \| ...)` instead. Because I want to keep this tightly focused and easy to review, this does not make any changes to other imports, documentation, etc., even within the changed lines. Those will come in future changes. Testing Done: Ran unit tests.	ztwqsqnlwluxkvoxvrlxvotwnqutuzxp

Issues

Description	From	Last Updated
line too long (81 > 79 characters) Column: 80 Error code: E501	reviewbot	June 12, 2026, 11:09 a.m.

flake8 failed.

JSHint passed.

flake8

reviewboard/notifications/email/backend.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (81 > 79 characters)

Column: 80
Error code: E501
```

Commits:

	Summary	ID
	Update typing: Union[...] This change updates legacy type hints that use the old `Union[...]` form to be `(... \| ...)` instead. Because I want to keep this tightly focused and easy to review, this does not make any changes to other imports, documentation, etc., even within the changed lines. Those will come in future changes. Testing Done: Ran unit tests.	ztwqsqnlwluxkvoxvrlxvotwnqutuzxp
	Update typing: Union[...] This change updates legacy type hints that use the old `Union[...]` form to be `(... \| ...)` instead. Because I want to keep this tightly focused and easy to review, this does not make any changes to other imports, documentation, etc., even within the changed lines. Those will come in future changes. Testing Done: Ran unit tests.	ztwqsqnlwluxkvoxvrlxvotwnqutuzxp

Diff:

Revision 2 (+280 -296)

Show changes

	reviewboard/accounts/models.py
	reviewboard/accounts/backends/base.py
	reviewboard/accounts/backends/standard.py
	reviewboard/actions/renderers.py
	reviewboard/datagrids/tests/test_users_list.py
	reviewboard/diffviewer/filediff_creator.py
	reviewboard/diffviewer/parser.py
	reviewboard/diffviewer/views.py
	38 more

Checks run (2 succeeded)

flake8 passed.

JSHint passed.