From 1decb4399d2fa9a5aaff197870cb365c0955dc63 Mon Sep 17 00:00:00 2001 From: xiongyuanwen-byted Date: Wed, 3 Jun 2026 14:46:05 +0800 Subject: [PATCH] test(sheets): drop redundant copyloopvar copy in required-flag parity test Go 1.22+ scopes the loop var per iteration, so `cmd, business := cmd, business` in TestBatchOp_RequiredFlagParity is a no-op that trips the repo's copyloopvar linter (same cleanup as 2132472). Behavior unchanged; 45 sub-tests still pass. --- shortcuts/sheets/batch_op_contract_test.go | 1 - 1 file changed, 1 deletion(-) diff --git a/shortcuts/sheets/batch_op_contract_test.go b/shortcuts/sheets/batch_op_contract_test.go index ea7f7fbac..16f99220a 100644 --- a/shortcuts/sheets/batch_op_contract_test.go +++ b/shortcuts/sheets/batch_op_contract_test.go @@ -878,7 +878,6 @@ func TestBatchOp_RequiredFlagParity(t *testing.T) { if len(business) == 0 { continue // only-locator commands (sheet-delete/hide/unhide/copy/filter-delete): nothing to omit } - cmd, business := cmd, business t.Run(cmd, func(t *testing.T) { t.Parallel() rawOp := map[string]interface{}{"shortcut": cmd, "input": sheetSel(cmd)}