fix linter errors

pivotal-cf · Jul 6, 2023 · a8380d7 · a8380d7
1 parent fb54765
commit a8380d7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/internal/builder/release_manifest_reader.go b/internal/builder/release_manifest_reader.go
@@ -2,6 +2,7 @@ package builder
 
 import (
 	"crypto/sha1"
+	"encoding/hex"
 	"fmt"
 	"github.com/go-git/go-billy/v5"
 	"github.com/go-git/go-billy/v5/osfs"
@@ -19,10 +20,6 @@ type ReleaseManifest struct {
 	StemcellVersion string `yaml:"-"`
 }
 
-type compiledPackage struct {
-	Stemcell string `yaml:"stemcell"`
-}
-
 type ReleaseManifestReader struct {
 	fs billy.Filesystem
 }
@@ -71,7 +68,7 @@ func (r ReleaseManifestReader) Read(releaseTarball string) (Part, error) {
 		return Part{}, err // NOTE: cannot replicate this error scenario in a test
 	}
 
-	outputReleaseManifest.SHA1 = fmt.Sprintf("%x", hash.Sum(nil))
+	outputReleaseManifest.SHA1 = hex.EncodeToString(hash.Sum(nil))
 
 	return Part{
 		File:     releaseTarball,

diff --git a/internal/builder/release_manifest_reader_test.go b/internal/builder/release_manifest_reader_test.go
@@ -220,7 +220,7 @@ version: 1.2.3
 					Expect(err).NotTo(HaveOccurred())
 
 					_, err = reader.Read(tarball.Name())
-					Expect(err).To(MatchError(fmt.Sprintf("failed to find release.MF in tarball")))
+					Expect(err).To(MatchError("failed to find release.MF in tarball"))
 				})
 			})