Skip to content

Commit

Permalink
Uncapitalize errors (#18915)
Browse files Browse the repository at this point in the history
- See: https://github.com/golang/go/wiki/CodeReviewComments#error-strings

Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
  • Loading branch information
Gusted and lunny committed Feb 26, 2022
1 parent fa0e2d6 commit 6d55a13
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 9 deletions.
6 changes: 3 additions & 3 deletions routers/web/repo/issue.go
Original file line number Diff line number Diff line change
Expand Up @@ -2557,7 +2557,7 @@ func updateAttachments(item interface{}, files []string) error {
case *models.Comment:
attachments = content.Attachments
default:
return fmt.Errorf("Unknown Type: %T", content)
return fmt.Errorf("unknown Type: %T", content)
}
for i := 0; i < len(attachments); i++ {
if util.IsStringInSlice(attachments[i].UUID, files) {
Expand All @@ -2575,7 +2575,7 @@ func updateAttachments(item interface{}, files []string) error {
case *models.Comment:
err = content.UpdateAttachments(files)
default:
return fmt.Errorf("Unknown Type: %T", content)
return fmt.Errorf("unknown Type: %T", content)
}
if err != nil {
return err
Expand All @@ -2587,7 +2587,7 @@ func updateAttachments(item interface{}, files []string) error {
case *models.Comment:
content.Attachments, err = repo_model.GetAttachmentsByCommentID(content.ID)
default:
return fmt.Errorf("Unknown Type: %T", content)
return fmt.Errorf("unknown Type: %T", content)
}
return err
}
Expand Down
10 changes: 5 additions & 5 deletions routers/web/repo/lfs.go
Original file line number Diff line number Diff line change
Expand Up @@ -120,14 +120,14 @@ func LFSLocks(ctx *context.Context) {
Shared: true,
}); err != nil {
log.Error("Failed to clone repository: %s (%v)", ctx.Repo.Repository.FullName(), err)
ctx.ServerError("LFSLocks", fmt.Errorf("Failed to clone repository: %s (%v)", ctx.Repo.Repository.FullName(), err))
ctx.ServerError("LFSLocks", fmt.Errorf("failed to clone repository: %s (%v)", ctx.Repo.Repository.FullName(), err))
return
}

gitRepo, err := git.OpenRepositoryCtx(ctx, tmpBasePath)
if err != nil {
log.Error("Unable to open temporary repository: %s (%v)", tmpBasePath, err)
ctx.ServerError("LFSLocks", fmt.Errorf("Failed to open new temporary repository in: %s %v", tmpBasePath, err))
ctx.ServerError("LFSLocks", fmt.Errorf("failed to open new temporary repository in: %s %v", tmpBasePath, err))
return
}
defer gitRepo.Close()
Expand All @@ -140,7 +140,7 @@ func LFSLocks(ctx *context.Context) {

if err := gitRepo.ReadTreeToIndex(ctx.Repo.Repository.DefaultBranch); err != nil {
log.Error("Unable to read the default branch to the index: %s (%v)", ctx.Repo.Repository.DefaultBranch, err)
ctx.ServerError("LFSLocks", fmt.Errorf("Unable to read the default branch to the index: %s (%v)", ctx.Repo.Repository.DefaultBranch, err))
ctx.ServerError("LFSLocks", fmt.Errorf("unable to read the default branch to the index: %s (%v)", ctx.Repo.Repository.DefaultBranch, err))
return
}

Expand Down Expand Up @@ -525,14 +525,14 @@ func LFSAutoAssociate(ctx *context.Context) {
for i, oid := range oids {
idx := strings.IndexRune(oid, ' ')
if idx < 0 || idx+1 > len(oid) {
ctx.ServerError("LFSAutoAssociate", fmt.Errorf("Illegal oid input: %s", oid))
ctx.ServerError("LFSAutoAssociate", fmt.Errorf("illegal oid input: %s", oid))
return
}
var err error
metas[i] = &models.LFSMetaObject{}
metas[i].Size, err = strconv.ParseInt(oid[idx+1:], 10, 64)
if err != nil {
ctx.ServerError("LFSAutoAssociate", fmt.Errorf("Illegal oid input: %s %v", oid, err))
ctx.ServerError("LFSAutoAssociate", fmt.Errorf("illegal oid input: %s %v", oid, err))
return
}
metas[i].Oid = oid[:idx]
Expand Down
2 changes: 1 addition & 1 deletion routers/web/repo/webhook.go
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ func getOrgRepoCtx(ctx *context.Context) (*orgRepoCtx, error) {
}, nil
}

return nil, errors.New("Unable to set OrgRepo context")
return nil, errors.New("unable to set OrgRepo context")
}

func checkHookType(ctx *context.Context) string {
Expand Down

0 comments on commit 6d55a13

Please sign in to comment.