Skip to content

Commit

Permalink
Removes "coverage" from default RUN. (letsencrypt#2750)
Browse files Browse the repository at this point in the history
Having "coverage" in the default RUN is leaving ".coverprofile" files
all over the src tree. This commit removes this task from the default
RUN list for local work. It is included in travis.yml and will still run
for CI or when specified explicitly with a command line env var
  override.
  • Loading branch information
cpu authored May 11, 2017
1 parent bd045b9 commit 1e57de2
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,9 @@ fi
# RUN variable with the ones you want (see .travis.yml for an example).
# Order doesn't matter. Note: godep-restore is specifically left out of the
# defaults, because we don't want to run it locally (would be too disruptive to
# GOPATH).
RUN=${RUN:-vet fmt migrations unit coverage integration errcheck}
# GOPATH). We also omit coverage by default on local runs because it generates
# artifacts on disk that aren't needed.
RUN=${RUN:-vet fmt migrations unit integration errcheck}

# The list of segments to hard fail on, as opposed to continuing to the end of
# the unit tests before failing.
Expand Down

0 comments on commit 1e57de2

Please sign in to comment.