From c47ba412d2e06f9ccb42471746f7c404947c156b Mon Sep 17 00:00:00 2001 From: Michael Brooks Date: Fri, 13 Nov 2026 11:24:55 -0600 Subject: [PATCH] refactor: use slices.Backward in internal/manifest/validate.go --- internal/manifest/validate.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/internal/manifest/validate.go b/internal/manifest/validate.go index 984e73ec..fcfcb6b5 100644 --- a/internal/manifest/validate.go +++ b/internal/manifest/validate.go @@ -100,9 +100,9 @@ func HandleConnectorApprovalRequired(ctx context.Context, clients *shared.Client } approvalRequiredErrors := slackerror.ErrorDetails{} - for i := len(slackError.Details) - 1; i >= 0; i-- { // Iterate in reverse to remove elements in place - if slackError.Details[i].Code == slackerror.ErrConnectorApprovalRequired { - approvalRequiredErrors = append(approvalRequiredErrors, slackError.Details[i]) + for i, detail := range slices.Backward(slackError.Details) { // Iterate in reverse to remove elements in place + if detail.Code == slackerror.ErrConnectorApprovalRequired { + approvalRequiredErrors = append(approvalRequiredErrors, detail) slackError.Details = slices.Delete(slackError.Details, i, i+1) } }