From a2e2177a9927997023beadad918be90eb6205fef Mon Sep 17 00:00:00 2001 From: Carlos Alexandro Becker Date: Wed, 24 Jan 2018 21:22:45 -0200 Subject: [PATCH] fix: linter errors --- build/build.go | 7 +++++++ pipeline/build/build_test.go | 7 ------- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/build/build.go b/build/build.go index eb797f7bf..b78549ee9 100644 --- a/build/build.go +++ b/build/build.go @@ -1,3 +1,4 @@ +// Package build provides the API for external builders package build import ( @@ -16,31 +17,37 @@ var ( lock sync.Mutex ) +// Register register a builder to a given lang func Register(lang string, builder Builder) { lock.Lock() builders[lang] = builder lock.Unlock() } +// For gets the previously register builder for the given lang func For(lang string) Builder { return builders[lang] } +// Options to be passed down to a builder type Options struct { Name, Path, Ext, Target string } +// Builder defines a builder type Builder interface { Default(build config.Build) config.Build Build(ctx *context.Context, build config.Build, options Options) error } +// Run runs a command within the given context and env func Run(ctx *context.Context, command, env []string) error { /* #nosec */ var cmd = exec.CommandContext(ctx, command[0], command[1:]...) var log = log.WithField("env", env).WithField("cmd", command) cmd.Env = append(cmd.Env, os.Environ()...) cmd.Env = append(cmd.Env, env...) + // TODO: improve debug here log.Debug("running") if out, err := cmd.CombinedOutput(); err != nil { log.WithError(err).Debug("failed") diff --git a/pipeline/build/build_test.go b/pipeline/build/build_test.go index 287368d36..cb7a49a5b 100644 --- a/pipeline/build/build_test.go +++ b/pipeline/build/build_test.go @@ -13,8 +13,6 @@ import ( "github.com/stretchr/testify/assert" ) -var emptyEnv []string - type fakeBuilder struct{} func (*fakeBuilder) Default(build config.Build) config.Build { @@ -217,8 +215,3 @@ func exists(file string) bool { _, err := os.Stat(file) return !os.IsNotExist(err) } - -func assertContainsError(t *testing.T, err error, s string) { - assert.Error(t, err) - assert.Contains(t, err.Error(), s) -}