From 117b9b974af928cad6d2185f3829102395274fe1 Mon Sep 17 00:00:00 2001 From: Matt Hammerly Date: Wed, 9 Apr 2025 09:39:47 -0700 Subject: [PATCH] fix: makefile build.requirements target had conditional in wrong place --- Makefile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index d9748d3dfb..822878a4d2 100644 --- a/Makefile +++ b/Makefile @@ -75,10 +75,10 @@ lint.check: build.requirements: # If make was given a different requirements tag, we assume a suitable image # was already built (e.g. by umbrella) and don't want to build this one. - ifneq (${REQUIREMENTS_TAG},${DEFAULT_REQS_TAG}) +ifneq (${REQUIREMENTS_TAG},${DEFAULT_REQS_TAG}) echo "Error: building api reqs image despite another being provided" exit 1 - endif +endif # if docker pull succeeds, we have already build this version of # requirements.txt. Otherwise, build and push a version tagged # with the hash of this requirements.txt