fix: improved artifact add logs

Makefile

@@ -4,6 +4,7 @@ TEST_OPTIONS?=
 
 # Install all the build and lint dependencies
 setup:
+	go get -u golang.org/x/tools/cmd/stringer
 	go get -u github.com/alecthomas/gometalinter
 	go get -u github.com/golang/dep/cmd/dep
 	go get -u github.com/pierrre/gotestcover
@@ -41,6 +42,7 @@ ci: test lint
 
 # Build a beta version of goreleaser
 build:
+	go generate ./...
 	go build
 .PHONY: build
 

internal/artifact/.gitignore

@@ -0,0 +1 @@
+type_string.go

internal/artifact/artifact.go

@@ -8,6 +8,7 @@ import (
 )
 
 // Type defines the type of an artifact
+//go:generate stringer -type=Type
 type Type int
 
 const (
@@ -57,16 +58,12 @@ func (artifacts Artifacts) List() []Artifact {
 	return artifacts.items
 }
 
-// Platform returns the platform string of a given artifact
-func (a Artifact) Platform() string {
-	return a.Goos + a.Goarch + a.Goarm
-}
-
 // GroupByPlatform groups the artifacts by their platform
 func (artifacts Artifacts) GroupByPlatform() map[string][]Artifact {
 	var result = map[string][]Artifact{}
 	for _, a := range artifacts.items {
-		result[a.Platform()] = append(result[a.Platform()], a)
+		plat := a.Goos + a.Goarch + a.Goarm
+		result[plat] = append(result[plat], a)
 	}
 	return result
 }
@@ -76,7 +73,9 @@ func (artifacts *Artifacts) Add(a Artifact) {
 	artifacts.lock.Lock()
 	defer artifacts.lock.Unlock()
 	log.WithFields(log.Fields{
-		"artifact": a,
+		"name": a.Name,
+		"path": a.Path,
+		"type": a.Type,
 	}).Info("added new artifact")
 	artifacts.items = append(artifacts.items, a)
 }

internal/artifact/artifact_test.go

@@ -1,12 +1,16 @@
 package artifact
 
 import (
+	"fmt"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
 	"golang.org/x/sync/errgroup"
 )
 
+// ensure Type implements the stringer interface...
+var _ fmt.Stringer = Type(0)
+
 func TestAdd(t *testing.T) {
 	var g errgroup.Group
 	var artifacts = New()

internal/buildtarget/buildtarget.go

@@ -28,6 +28,7 @@ func (t Target) Env() []string {
 }
 
 func (t Target) String() string {
+	// TODO: maybe replace this as suggested to OS_ArchArm?
 	return fmt.Sprintf("%v%v%v", t.OS, t.Arch, t.Arm)
 }
 

pipeline/archive/archive.go

@@ -67,7 +67,7 @@ func (Pipe) Default(ctx *context.Context) error {
 }
 
 func create(ctx *context.Context, artifacts []artifact.Artifact) error {
-	var format = archiveformat.For(ctx, artifacts[0].Platform())
+	var format = archiveformat.For(ctx, artifacts[0].Goos)
 	folder, err := nametemplate.Apply(ctx, artifacts[0], ctx.Config.ProjectName)
 	if err != nil {
 		return err