From 9bd9a33ca4fe9a6bbff4addf1e98fd05d1e8ea2c Mon Sep 17 00:00:00 2001
From: Jonathan Yu <jawnsy@redhat.com>
Date: Wed, 28 Sep 2016 12:54:04 -0700
Subject: [PATCH] Remove redundant build steps

`make install.tools` already installs `gometalinter` (as can be seen in
the logs), so the subsequent `go get` steps are unnecessary.

This change also reorders the build directives according to execution
order: https://docs.travis-ci.com/user/customizing-the-build#The-Build-Lifecycle
and replaces the "before_install" step with an override of the default
"install" procedure (which we were suppressing anyway with install: true)

Signed-off-by: Jonathan Yu <jawnsy@redhat.com>
---
 .travis.yml | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/.travis.yml b/.travis.yml
index 2c3c9a4b..ecafe37b 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -8,16 +8,12 @@ sudo: required
 services:
   - docker
 
+install:
+  - make install.tools
+
 before_script:
   - export PATH=$HOME/gopath/bin:$PATH
 
-before_install:
-  - make install.tools
-  - go get -u github.com/alecthomas/gometalinter
-  - gometalinter --install --update
-
-install: true
-
 script:
   - make .gitvalidation
   - make lint