fix: fix miss makezero bug (#6971)

alingse · web-flow · commit 389ecf614a52 · 2024-05-28T15:10:29.000+02:00
Signed-off-by: alingse &lt;alingse@foxmail.com&gt;
diff --git a/engine/api/migrate/migrate_organization.go b/engine/api/migrate/migrate_organization.go
@@ -33,7 +33,7 @@ func GetOrganizationUsersToMigrate(ctx context.Context, db *gorp.DbMap) ([]UserO
 		return nil, err
 	}
 
-	userIds := make([]string, len(allUsers))
+	userIds := make([]string, 0, len(allUsers))
 	mapUsers := make(map[string]*sdk.AuthentifiedUser)
 	for i := range allUsers {
 		u := &allUsers[i]
diff --git a/engine/hooks/trigger_workflow.go b/engine/hooks/trigger_workflow.go
@@ -84,7 +84,7 @@ func (s *Service) triggerWorkflows(ctx context.Context, hre *sdk.HookRepositoryE
 					wh.Status = sdk.HookEventWorkflowStatusSkipped
 				} else {
 					// Query params to select the right workflow version to run
-					mods := make([]cdsclient.RequestModifier, 2)
+					mods := make([]cdsclient.RequestModifier, 0, 2)
 					mods = append(mods, cdsclient.WithQueryParameter("ref", wh.Ref), cdsclient.WithQueryParameter("commit", wh.Commit))
 
 					runRequest := sdk.V2WorkflowRunHookRequest{

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ func GetOrganizationUsersToMigrate(ctx context.Context, db *gorp.DbMap) ([]UserO`
`33`	`33`	`return nil, err`
`34`	`34`	`}`
`35`	`35`
`36`		`- userIds := make([]string, len(allUsers))`
	`36`	`+ userIds := make([]string, 0, len(allUsers))`
`37`	`37`	`mapUsers := make(map[string]*sdk.AuthentifiedUser)`
`38`	`38`	`for i := range allUsers {`
`39`	`39`	`u := &allUsers[i]`