fix

Author: wxiaoguang

cmd/embedded.go

@@ -301,8 +301,7 @@ func compileCollectPatterns(args []string) (_ []glob.Glob, err error) {
 	}
 	pat := make([]glob.Glob, len(args))
 	for i := range args {
-		pat[i], err = glob.Compile(args[i], '/')
-		if err != nil {
+		if pat[i], err = glob.Compile(args[i], '/'); err != nil {
 			return nil, fmt.Errorf("invalid glob patterh %q: %w", args[i], err)
 		}
 	}

models/migrations/v1_14/v165.go

@@ -39,7 +39,7 @@ func ConvertHookTaskTypeToVarcharAndTrim(x *xorm.Engine) error {
 		return err
 	}
 
-	// Webhook: string `xorm:"VARCHAR(16) index"`
+	// Webhook: Type string `xorm:"VARCHAR(16) index"`
 	if err := base.ModifyColumn(x, "webhook", &schemas.Column{
 		Name: "type",
 		SQLType: schemas.SQLType{

modules/graceful/manager_windows.go

@@ -41,7 +41,7 @@ func (g *Manager) start() {
 	// Make SVC process
 	run := svc.Run
 
-	isAnInteractiveSession, err := svc.IsAnInteractiveSession() //nolint:staticcheck // must IsAnInteractiveSession because IsWindowsService has a different permissions profile
+	isAnInteractiveSession, err := svc.IsAnInteractiveSession() //nolint:staticcheck // must use IsAnInteractiveSession because IsWindowsService has a different permissions profile
 	if err != nil {
 		log.Error("Unable to ascertain if running as an Windows Service: %v", err)
 		return

modules/templates/scopedtmpl/scopedtmpl.go

@@ -109,7 +109,7 @@ type htmlTemplate struct {
 
 type textTemplateCommon struct {
 	_/*tmpl*/ map[string]*template.Template
-	_/*muTmpl*/ sync.RWMutex //
+	_/*muTmpl*/ sync.RWMutex
 	_/*option*/ struct {
 		missingKey int
 	}

services/pull/merge.go

@@ -325,7 +325,7 @@ func handleCloseCrossReferences(ctx context.Context, pr *issues_model.PullReques
 }
 
 // doMergeAndPush performs the merge operation without changing any pull information in database and pushes it up to the base repository
-func doMergeAndPush(ctx context.Context, pr *issues_model.PullRequest, doer *user_model.User, mergeStyle repo_model.MergeStyle, expectedHeadCommitID, message string, pushTrigger repo_module.PushTrigger) (string, error) { //nolint:unparam // non error result is never used
+func doMergeAndPush(ctx context.Context, pr *issues_model.PullRequest, doer *user_model.User, mergeStyle repo_model.MergeStyle, expectedHeadCommitID, message string, pushTrigger repo_module.PushTrigger) (string, error) { //nolint:unparam // non-error result is never used
 	// Clone base repo.
 	mergeCtx, cancel, err := createTemporaryRepoForMerge(ctx, pr, doer, expectedHeadCommitID)
 	if err != nil {