Generate diffs between base and tip commits

Review Request #10130 — Created Aug. 27, 2018 and updated

brennie
Review Board
release-4.0.x
10124
5af6c50...
reviewboard

The get_diff_files utility can now generate a list of files to diff
that exist between two commits in a diffset. This allows us to generate
diffs between two commits (and therefore to see the diff of any
individual commit). There is currently no UI support for this behaviour,
but that is comming in a future patch.

Ran unit tests.

  • 0
  • 0
  • 12
  • 0
  • 12
Description From Last Updated
Checks run (1 failed, 1 succeeded)
flake8 failed.
JSHint passed.

flake8

brennie
chipx86
  1. 
      
  2. reviewboard/diffviewer/diffutils.py (Diff revision 2)
     
     

    Missing space between the backtick and "diffsets".

  3. reviewboard/diffviewer/diffutils.py (Diff revision 2)
     
     

    Here, too.

  4. reviewboard/diffviewer/diffutils.py (Diff revision 2)
     
     
     

    Missing blank line.

  5. reviewboard/diffviewer/tests/test_diffutils.py (Diff revision 2)
     
     
     
     

    No blank line needed here.

  6. reviewboard/diffviewer/tests/test_diffutils.py (Diff revision 2)
     
     
     
     
     
     

    Given the repeated complexity of this, it would be nice to make this whole expected_results building into a private utility method.

  7. 
      
brennie
chipx86
  1. 
      
  2. reviewboard/diffviewer/diffutils.py (Diff revision 3)
     
     
     

    Redundant "diffsets" in this sentence.

  3. reviewboard/diffviewer/diffutils.py (Diff revision 3)
     
     
     

    Same here.

  4. reviewboard/diffviewer/diffutils.py (Diff revision 3)
     
     

    Can you compare explicitly against 0?

  5. reviewboard/diffviewer/diffutils.py (Diff revision 3)
     
     

    Here, too.

  6. Should be .. code-block:: python (note the ::).

  7. reviewboard/diffviewer/tests/test_diffutils.py (Diff revision 3)
     
     
     
     
     
     
     

    This doesn't match.

    1. Weird, I must've missed committing the docs change because I have it locally.

  8. 
      
brennie
Review request changed

Change Summary:

Addressed feedback.

Commit:

-9d8e07f93a138d75da763d2474536df69d9a0c22
+5af6c501da5c8b9a449e36bddb2b4af28ece2f78

Diff:

Revision 4 (+352 -12)

Show changes

Checks run (2 succeeded)

flake8 passed.
JSHint passed.
Loading...