remove delete of extra CRDs dir

This was intended to help clean up the extra CRDs dir which was created by mistake in #239. However, a different makefile-modules PR (#240) meant that a manual upgrade was required for downstream repos using makefile-modules anyway. As such, this is just error-prone and liable to break ci (it should've been `rm -rf` anyway). Signed-off-by: Ashley Davis <[email protected]>
cert-manager · Jan 29, 2025 · 92757ad · 92757ad
1 parent 99dd04f
commit 92757ad
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/modules/helm/crds.mk b/modules/helm/crds.mk
@@ -50,9 +50,8 @@ generate-crds: | $(NEEDS_CONTROLLER-GEN) $(NEEDS_YQ)
 	# TODO(@SgtCoDFish): This is a temporary fix for an accidentally created nested crds directory
 	# in the PR: https://github.com/cert-manager/makefile-modules/pull/239
 	# Once this has been run on every repo with the mistaken folder, this can be removed
-	rm -r $(crds_dir)/crds
+	rm -rf $(crds_dir)/crds
 
-	rm -rf $(crds_gen_temp)
 	mkdir -p $(crds_gen_temp)
 
 	$(CONTROLLER-GEN) crd \