Skip to content

Commit

Permalink
Merge pull request #1302 from dgageot/dont-test-on-redeploy
Browse files Browse the repository at this point in the history
Don’t run tests if nothing was built
  • Loading branch information
dgageot committed Nov 21, 2018
2 parents 70de84e + 86f47e6 commit ae1ed38
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions pkg/skaffold/runner/runner.go
Original file line number Diff line number Diff line change
Expand Up @@ -299,10 +299,6 @@ func (r *SkaffoldRunner) Dev(ctx context.Context, out io.Writer, artifacts []*la
return nil
}
case changed.needsRedeploy:
if err := r.Test(ctx, out, r.builds); err != nil {
logrus.Warnln("Skipping Deploy due to failed tests:", err)
return nil
}
if _, err := r.Deploy(ctx, out, r.builds); err != nil {
logrus.Warnln("Skipping Deploy due to error:", err)
return nil
Expand Down

0 comments on commit ae1ed38

Please sign in to comment.