From 6d55a132a5b706e05cfdd02f8ff597972665c554 Mon Sep 17 00:00:00 2001 From: Gusted Date: Sat, 26 Feb 2022 12:15:32 +0000 Subject: [PATCH] Uncapitalize errors (#18915) - See: https://github.com/golang/go/wiki/CodeReviewComments#error-strings Co-authored-by: Lunny Xiao --- routers/web/repo/issue.go | 6 +++--- routers/web/repo/lfs.go | 10 +++++----- routers/web/repo/webhook.go | 2 +- 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a/routers/web/repo/issue.go b/routers/web/repo/issue.go index 952135eaf5..d0c3d3325a 100644 --- a/routers/web/repo/issue.go +++ b/routers/web/repo/issue.go @@ -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) { @@ -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 @@ -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 } diff --git a/routers/web/repo/lfs.go b/routers/web/repo/lfs.go index 2cb4330c08..d2d62786fe 100644 --- a/routers/web/repo/lfs.go +++ b/routers/web/repo/lfs.go @@ -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() @@ -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 } @@ -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] diff --git a/routers/web/repo/webhook.go b/routers/web/repo/webhook.go index fb984de7f5..76ebd1c485 100644 --- a/routers/web/repo/webhook.go +++ b/routers/web/repo/webhook.go @@ -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 {