go/goreleaser
go/goreleaser
1
0
Fork 0
mirror of https://github.com/goreleaser/goreleaser.git synced 2025-03-17 20:47:50 +02:00
Code Issues Releases Activity

Revert "feat: git diff when dirty (#1178)"

Browse Source 
This reverts commit 39d07c375df531aa0c55df868d26f8dcc4bf37e1.
This commit is contained in:
Carlos Alexandro Becker 2019-10-20 11:10:31 -03:00
parent 7151b99bed
commit 4e71977e34
No known key found for this signature in database
GPG Key ID: E61E2F7DC14AB940
3 changed files with 4 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
go.sum
View File

@ -328,7 +328,6 @@ github.com/src-d/gcfg v1.4.0/go.mod h1:p/UMsR43ujA89BJY9duynAwIpvqEujIH/jFlfL7jW
github.com/steveyen/gtreap v0.0.0-20150807155958-0abe01ef9be2/go.mod h1:mjqs7N0Q6m5HpR7QfXVBZXZWSqTjQLeTujjA/xUp2uw=
github.com/stretchr/objx v0.1.0/go.mod h1:HFkY916IF+rwdDfMAkV7OtwuqBVzrE8GR6GFx+wExME=
github.com/stretchr/objx v0.1.1/go.mod h1:HFkY916IF+rwdDfMAkV7OtwuqBVzrE8GR6GFx+wExME=
github.com/stretchr/objx v0.2.0 h1:Hbg2NidpLE8veEBkEZTL3CvlkUIVzuU9jDplZO54c48=
github.com/stretchr/objx v0.2.0/go.mod h1:qt09Ya8vawLte6SNmTgCsAVtYtaKzEcn8ATUoHMkEqE=
github.com/stretchr/testify v1.2.2/go.mod h1:a8OnRcib4nhh0OaRAV+Yts87kKdq0PP7pXfy6kDkUVs=
github.com/stretchr/testify v1.3.0/go.mod h1:M5WIy9Dh21IEIfnGCwXGc5bZfKNJtfHm1UVUgZn+9EI=

21
internal/pipe/git/git.go
View File

@ -99,10 +99,11 @@ func validate(ctx *context.Context) error {
if ctx.SkipValidate {
return pipe.ErrSkipValidateEnabled
}
if hasDiff() {
return diffErr()
out, err := git.Run("status", "--porcelain")
if strings.TrimSpace(out) != "" || err != nil {
return ErrDirty{status: out}
}
_, err := git.Clean(git.Run("describe", "--exact-match", "--tags", "--match", ctx.Git.CurrentTag))
_, err = git.Clean(git.Run("describe", "--exact-match", "--tags", "--match", ctx.Git.CurrentTag))
if err != nil {
return ErrWrongRef{
commit: ctx.Git.Commit,
@ -112,20 +113,6 @@ func validate(ctx *context.Context) error {
return nil
}
func diffErr() error {
out, err := git.Run("diff")
if err != nil {
// in theory this will never happen...
return ErrDirty{status: err.Error()}
}
return ErrDirty{status: out}
}
func hasDiff() bool {
out, err := git.Run("status", "--porcelain")
return strings.TrimSpace(out) != "" || err != nil
}
func getShortCommit() (string, error) {
return git.Clean(git.Run("show", "--format='%h'", "HEAD", "-q"))
}

1
internal/pipe/git/git_test.go
View File

@ -91,7 +91,6 @@ func TestDirty(t *testing.T) {
err = Pipe{}.Run(context.New(config.Project{}))
assert.Error(t, err)
assert.Contains(t, err.Error(), "git is currently in a dirty state")
assert.Contains(t, err.Error(), "+lorem ipsum")
})
t.Run("skip validate is set", func(t *testing.T) {
ctx := context.New(config.Project{})