Skip to content

Commit 60610ba

Browse files
authored
Merge pull request #48 from codecrafters-io/andy/tweak
Tweak message when buildpack is already up to date
2 parents 4337039 + be153a3 commit 60610ba

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

internal/commands/test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -189,8 +189,8 @@ func pushBranchToRemote(tmpDir string, remoteName string) error {
189189

190190
func wrapError(err error, output []byte, msg string) error {
191191
if _, ok := err.(*exec.ExitError); ok {
192-
return fmt.Errorf("add all files: %s", output)
192+
return fmt.Errorf("%s: %s. Error: %w", msg, output, err)
193193
}
194194

195-
return fmt.Errorf("add all files: %w", err)
195+
return fmt.Errorf("%s: %w", msg, err)
196196
}

internal/commands/update_buildpack.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,7 @@ func UpdateBuildpackCommand(ctx context.Context) (err error) {
9292

9393
if currentBuildpackSlug == latestBuildpack.Slug {
9494
fmt.Printf("Buildpack is already up to date (%s)\n", currentBuildpackSlug)
95+
fmt.Println("Let us know at hello@codecrafters.io if you’d like us to upgrade the supported buildpack version.")
9596
return nil
9697
}
9798

0 commit comments

Comments
 (0)