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: Closed (submitted)

Change Summary:

Pushed to release-1.0.x (794ba24)
Loading...