Limit multi-commit review requests to repositories that support them

Review Request #9883 — Created April 20, 2018 and submitted

Information

Review Board
release-4.0.x
69fdb9d...

Reviewers

SCMTools now have a supports_history flag that indicates whether or
not they support multi-commit review requests. The API now prevents
creating a review request with history when the underlying SCMTool does
not support it.

For now, only Git will have the support enabled (althrough incomplete)
until the overall DVCS support is feature complete and then support for
other SCMs can be added.

Ran unit tests.

Description From Last Updated

F811 redefinition of unused 'test_post_create_with_history_enabled_unsupported_tool' from line 1263

reviewbotreviewbot

Looks like this line wasn't supposed to be deleted.

daviddavid
brennie
Review request changed
Change Summary:

More tests.

Commit:
1266b03c5b7dbf1eeb165753079f317b74a08fe2
640d178ff1d34c0f420807a89b1c857e26cbc894

Checks run (1 failed, 1 succeeded)

flake8 failed.
JSHint passed.

flake8

brennie
david
  1. 
      
  2. reviewboard/scmtools/core.py (Diff revision 3)
     
     
    Show all issues

    Looks like this line wasn't supposed to be deleted.

  3. 
      
brennie
brennie
david
  1. Ship It!
  2. 
      
chipx86
  1. Ship It!
  2. 
      
brennie
chipx86
  1. Ship It!
  2. 
      
brennie
Review request changed
Status:
Completed
Change Summary:
Pushed to release-4.0.x (57c01fc)