Split diffviewer models into multiple files

Review Request #9597 — Created Feb. 7, 2018 and submitted

brennie
Review Board
release-4.0.x
776363f...
reviewboard

The diffviewer/models.py file was getting quite large (~750 lines) and
is going to be growing due to some new model additions for DVCS support.
It was about time it got cleaned up.

This change also cleans up some documentation to bring it in line with
our current standards.

Ran unit tests

  • 1
  • 0
  • 1
  • 0
  • 2
Description From Last Updated
I didn't review this change before it went in :( I was going to ask if you could rename diff_set.py ... chipx86 chipx86
Checks run (1 failed, 1 succeeded)
flake8 failed.
JSHint passed.

flake8

brennie
david
  1. Ship It!
  2. 
      
brennie
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-4.0.x (0a42896)
chipx86
  1. 
      
  2. I didn't review this change before it went in :( I was going to ask if you could rename diff_set.py to diffset.py, since the latter is the standard lowercase spelling for this.

  3. 
      
Loading...