flake8
-
reviewboard/reviews/builtin_fields.py (Diff revision 1) Show all issues
Review Request #10522 — Created April 9, 2019 and updated
The CommitListField now renders change description entries for the field
more similarily to its regular field view. Previously, it would just
show all old commits as removed and all new commits as added, but now it
performs a history diff between the old and new commits and renders that
instead.The new template supports rendering commit modifications, however
commit_utils.diff_histories
presently cannot generate history diffs
that contain modifications (only additions, deletions, and unmodified
entries).
Ran unit tests.
Summary | ID |
---|---|
1d5306bd97fe94ee92cb442c2d35d09ad8928f67 |
Description | From | Last Updated |
---|---|---|
Can you add screenshots? |
|
|
F841 local variable 'to_expand' is assigned to but never used |
![]() |
|
F841 local variable 'to_expand' is assigned to but never used |
![]() |
|
Instead of defining yet another color constant for me to replace, how about just defining the background color for .changed … |
|
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+462 -102) |
reviewboard/reviews/builtin_fields.py (Diff revision 2) |
---|
F841 local variable 'to_expand' is assigned to but never used
Testing Done: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Commits: |
|
|||||||
Diff: |
Revision 3 (+458 -102) |
attach unit tests
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 4 (+456 -102) |
Added screenshots.
NOTE:
In the "Modifying a single commit (after)" screenshot,
diff_histories
cannot detect commit modifications, so this is a screenshot of what it would look like if it could.
Description: |
|
||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Added Files: |
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 5 (+472 -104) |
|||||||
Removed Files: |
||||||||
Added Files: |
reviewboard/static/rb/css/defs.less (Diff revision 5) |
---|
Instead of defining yet another color constant for me to replace, how about just defining the background color for
.changed td
as a variable and then usingdarken(..., ...)
?