Merge pull request #180 from Financial-Times/allow_template_folders

check result not empty
Financial-Times · May 23, 2017 · 80c1c52 · 80c1c52
2 parents 59b0923 + 3319ada
commit 80c1c52
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/Makefile b/Makefile
@@ -191,8 +191,8 @@ VERIFY_MSG_NO_PA11Y = "\n**** Error ****\nIt looks like your code is user-facing
 #check if project has HTML and missing make a11y command
 #check if project has demo app if there's a make a11y command
 _verify_pa11y_testable:
-	@if [ "$(IS_USER_FACING)" ] && [ -z $(MAKEFILE_HAS_A11Y) ] && [ ! ${IGNORE_A11Y} ]; then (printf $(VERIFY_MSG_NO_PA11Y) && exit 1); fi
-	@if [ "$(IS_USER_FACING)" ] && [ ! -d server ] && [ ! -f demos/app.js ]; then (echo $(VERIFY_MSG_NO_DEMO) && exit 1); fi
+	@if [ ! -z "$(IS_USER_FACING)" ] && [ -z $(MAKEFILE_HAS_A11Y) ] && [ ! ${IGNORE_A11Y} ]; then (printf $(VERIFY_MSG_NO_PA11Y) && exit 1); fi
+	@if [ ! -z "$(IS_USER_FACING)" ] && [ ! -d server ] && [ ! -f demos/app.js ]; then (echo $(VERIFY_MSG_NO_DEMO) && exit 1); fi
 	@$(DONE)
 
 _run_pa11y: