Skip to content

Commit 15c9b22

Browse files
authored
fix(ci): docker rm requires at least 1 argument (#518)
* fix(ci): docker rm requires at least 1 argument * fix: replace $$ with $()
1 parent 2c2a4a3 commit 15c9b22

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

Makefile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -249,9 +249,9 @@ cleanup-integration-test: $(KIND)
249249
$(KIND) delete cluster -n test-kind-cluster || true
250250
@echo "### Removing docker containers and images"
251251
$(eval CONTAINERS := $(shell $(OCI_BIN) ps --format '{{.Names}}' | grep 'integration-'))
252-
$(if $(strip $(CONTAINERS)),$(OCI_BIN) rm -f $$CONTAINERS,@echo "No integration test containers to remove")
252+
$(if $(strip $(CONTAINERS)),$(OCI_BIN) rm -f $(CONTAINERS),@echo "No integration test containers to remove")
253253
$(eval IMAGES := $(shell $(OCI_BIN) images --format '{{.Repository}}:{{.Tag}}' | grep 'hatest-'))
254-
$(if $(strip $(CONTAINERS)),$(OCI_BIN) rmi -f $$IMAGES,@echo "No integration test images to remove")
254+
$(if $(strip $(IMAGES)),$(OCI_BIN) rmi -f $(IMAGES),@echo "No integration test images to remove")
255255

256256
.PHONY: run-integration-test
257257
run-integration-test:

0 commit comments

Comments
 (0)