• 
      

    Move Travis CI checkout and patching to `before_install` section.

    Review Request #10976 — Created March 27, 2020 and submitted

    Information

    rbintegrations
    release-1.0.x
    cec054b...

    Reviewers

    In repos where dependencies are changing between branches, having the
    checkout and patch steps run as part of the Travis CI script would
    sometimes fail (for example, attempting to do builds of djblets on
    release-1.0.x failed in install because it was trying to get
    django-evolution 2.0, as required in the master branch).

    Doing the checkout and patch in before_install makes a lot more sense,
    because then we've got the full change in the working directory before
    doing anything.

    Ran a build of a change from djblets-1.0.x and saw that it got past the
    initial install/setup steps, where before it didn't.

    Description From Last Updated

    Actually, can you add some unit tests for this?

    chipx86chipx86

    E501 line too long (81 > 79 characters)

    reviewbotreviewbot
    Checks run (1 failed, 1 succeeded)
    flake8 failed.
    JSHint passed.

    flake8

    david
    chipx86
    1. Ship It!
    2. 
        
    chipx86
    1. 
        
    2. Show all issues

      Actually, can you add some unit tests for this?

    3. 
        
    david
    david
    chipx86
    1. Ship It!
    2. 
        
    david
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-1.0.x (794ba24)