• 
      

    Improve output when scanning for a usable client.

    Review Request #12512 — Created Aug. 8, 2022 and submitted

    Information

    RBTools
    release-4.x

    Reviewers

    This rewords some of the output when trying to scan for a usable client
    but failing to find anything. It aims to add context to the messages,
    helping to see the input (repository type or URL) that led to the error,
    and to add instructions to help resolve the issue (saying to check for a
    .reviewboardrc, what it should contain, and pointing users to
    rbt setup-repo).

    The work here is just a base for upcoming work on this logic to clean
    things up and to provide error reporting that works with the upcoming
    formal SCMClient dependency support.

    Tested all the error/warning conditions, verifying the output.

    Summary ID
    Improve output when scanning for a usable client.
    This rewords some of the output when trying to scan for a usable client but failing to find anything. It aims to add context to the messages, helping to see the input (repository type or URL) that led to the error, and to add instructions to help resolve the issue (saying to check for a .reviewboardrc, what it should contain, and pointing users to `rbt setup-repo`). The work here is just a base for upcoming work on this logic to clean things up and to provide error reporting that works with the upcoming formal `SCMClient` dependency support.
    39a1b35b05214c932835b8be18befc95280b7a0d
    Description From Last Updated

    Can we say "could not" instead of "couldn't"? I feel like it reads better.

    daviddavid

    Same here.

    daviddavid

    This can use single quotes.

    daviddavid
    david
    1. 
        
    2. rbtools/clients/__init__.py (Diff revision 1)
       
       
      Show all issues

      Can we say "could not" instead of "couldn't"? I feel like it reads better.

      1. Updating these to use "was not" to be in line with other logs here.

    3. rbtools/clients/__init__.py (Diff revision 1)
       
       
      Show all issues

      Same here.

    4. rbtools/clients/__init__.py (Diff revision 1)
       
       
       
       
       
       
       
       
      Show all issues

      This can use single quotes.

    5. 
        
    chipx86
    david
    1. Ship It!
    2. 
        
    chipx86
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-4.x (9b34e23)