From 34b4b106c67de64e0193b8b3cef17fc04a103dcc Mon Sep 17 00:00:00 2001 From: Kir Kolyshkin Date: Wed, 19 Aug 2020 15:53:51 -0700 Subject: [PATCH] tests/int: alt fix for shellcheck SC2034 Instead of ignoring it, use a throwaway variable _ (yes, just like in Go). Signed-off-by: Kir Kolyshkin --- tests/integration/checkpoint.bats | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/tests/integration/checkpoint.bats b/tests/integration/checkpoint.bats index 4cb4b648e..c7debd6cb 100644 --- a/tests/integration/checkpoint.bats +++ b/tests/integration/checkpoint.bats @@ -61,8 +61,7 @@ function simple_cr() { testcontainer test_busybox running - # shellcheck disable=SC2034 - for i in $(seq 2); do + for _ in $(seq 2); do # checkpoint the running container runc --criu "$CRIU" checkpoint --work-path ./work-dir test_busybox grep -B 5 Error ./work-dir/dump.log || true @@ -243,8 +242,7 @@ function simple_cr() { testcontainer test_busybox running - # shellcheck disable=SC2034 - for i in $(seq 2); do + for _ in $(seq 2); do # checkpoint the running container; this automatically tells CRIU to # handle the network namespace defined in config.json as an external runc --criu "$CRIU" checkpoint --work-path ./work-dir test_busybox