Skip to content

Commit d451bb4

Browse files
authored
Merge pull request #71 from github/move-error-check
Move an error check to a more correct location.
2 parents 4e0758f + 48a8250 commit d451bb4

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

internal/push/push.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -324,15 +324,15 @@ func (pushService *pushService) createOrUpdateReleaseAsset(release *github.Repos
324324
}
325325
log.Debugf("Uploading release asset %s...", assetPathStat.Name())
326326
assetFile, err := os.Open(pushService.cacheDirectory.AssetPath(release.GetTagName(), assetPathStat.Name()))
327+
if err != nil {
328+
return errors.Wrap(err, "Error opening release asset.")
329+
}
327330
defer assetFile.Close()
328331
progressReader := &ioprogress.Reader{
329332
Reader: assetFile,
330333
Size: assetPathStat.Size(),
331334
DrawFunc: ioprogress.DrawTerminalf(os.Stderr, ioprogress.DrawTextFormatBytes),
332335
}
333-
if err != nil {
334-
return errors.Wrap(err, "Error opening release asset.")
335-
}
336336
_, _, err = pushService.uploadReleaseAsset(release, assetPathStat, progressReader)
337337
if err != nil {
338338
return errors.Wrap(err, "Error uploading release asset.")

0 commit comments

Comments
 (0)