add --guess-branch support (clearcase implementation only)

Review Request #6515 — Created Oct. 28, 2014 and discarded

Information

RBTools
release-0.6.x

Reviewers

Implementation provided when revisions are UCM ClearCase activity and base ClearCase branch (brtype:)

cf this thread
https://groups.google.com/forum/#!topic/reviewboard-dev/gGWV2DQhgZ4

can post successfully this request with --guess-branch=no

can post successfully review request --guess-branch=yes from clearcase scm with revisions brtype:branch against reviewboard server 1.7.6 (current branch filled)

can post successfully review request --guess-branch=yes from ucm clearcase activity with revisions activity:activity against reviewboard server 1.7.6 (deliver stream filled)

Description From Last Updated

Col: 41 E231 missing whitespace after ','

reviewbotreviewbot

Col: 80 E501 line too long (80 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (80 > 79 characters)

reviewbotreviewbot

(reply to myself as a reminder) should raise "NotImplementedError" so that this exception can be caught as expected when using …

DE delyn
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        rbtools/clients/clearcase.py
        rbtools/commands/post.py
        rbtools/clients/__init__.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        rbtools/clients/clearcase.py
        rbtools/commands/post.py
        rbtools/clients/__init__.py
    
    
  2. rbtools/clients/clearcase.py (Diff revision 1)
     
     
    Show all issues
    Col: 41
     E231 missing whitespace after ','
    
  3. rbtools/clients/clearcase.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (80 > 79 characters)
    
  4. rbtools/clients/clearcase.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (80 > 79 characters)
    
  5. 
      
DE
reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        rbtools/clients/clearcase.py
        rbtools/commands/post.py
        rbtools/clients/__init__.py
    
    
    
    Tool: Pyflakes
    Processed Files:
        rbtools/clients/clearcase.py
        rbtools/commands/post.py
        rbtools/clients/__init__.py
    
    
  2. 
      
DE
  1. 
      
  2. rbtools/clients/clearcase.py (Diff revision 2)
     
     
    Show all issues

    (reply to myself as a reminder) should raise "NotImplementedError" so that this exception can be caught as expected when using base clearcase lbtype comparison

  3. 
      
DE
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        rbtools/clients/clearcase.py
        rbtools/commands/post.py
        rbtools/clients/__init__.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        rbtools/clients/clearcase.py
        rbtools/commands/post.py
        rbtools/clients/__init__.py
    
    
  2. 
      
DE
Review request changed
Status:
Discarded