Split up reviews/views.py

Review Request #12684 — Created Oct. 14, 2022 and submitted — Latest diff uploaded

Information

Review Board
release-6.x

Reviewers

The main reviews/views.py file was pretty huge, and was very difficult
to navigate. This change splits it up into a bunch of smaller files.

There are a couple trivial changes (adding some asserts, defining a
name attribute on BugTracker) in order to make type checkers a
bit happier, but otherwise this is purely moving things around.

Ran unit tests.

Changes between revision 2 and 3

orig
1
2
3
4

Commits

Summary ID Author
Split up reviews/views.py
The main reviews/views.py file was pretty huge, and was very difficult to navigate. This change splits it up into a bunch of smaller files. There are a couple trivial changes (adding some asserts, defining a `name` attribute on `BugTracker`) in order to make type checkers a bit happier, but otherwise this is purely moving things around. Testing Done: Ran unit tests.
8ff2c48a93239f260898c7e5ecf5982d4b3e73d6 David Trowbridge
Split up reviews/views.py
The main reviews/views.py file was pretty huge, and was very difficult to navigate. This change splits it up into a bunch of smaller files. There are a couple trivial changes (adding some asserts, defining a `name` attribute on `BugTracker`) in order to make type checkers a bit happier, but otherwise this is purely moving things around. Testing Done: Ran unit tests.
e6edaf78ad3ac01217b7150cf92b165940385aaf David Trowbridge
reviewboard/reviews/views/__init__.py
Loading...