Remove does_set (because it will be implemented in doe-suite)

Make DOES_RESULTS_DIR public
nicolas-kuechler · Aug 9, 2024 · 30433bc · 30433bc
1 parent a81e853
commit 30433bc
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/Makefile b/Makefile
@@ -379,9 +379,3 @@ docs-build: install
 
 docs: docs-build
 	@open docs/build/html/index.html
-
-
-## Checks does_config_dir is set
-does_set:
-	@if [ ! -d "$(does_config_dir)" ]; then echo "DOES_PROJECT_DIR is not set, please set environment variables"; exit 1; fi
-
diff --git a/doespy/doespy/util.py b/doespy/doespy/util.py
@@ -94,8 +94,8 @@ def get_suite_roles_dir():
 
 def get_results_dir():
     prj_dir = get_project_dir()
-    results_dir_name = os.environ.get("DOES_RESULTS_DIR", "doe-suite-results")
-    results_dir = os.path.join(prj_dir, results_dir_name)
+    results_dir_name = "doe-suite-results"
+    results_dir = os.environ.get("DOES_RESULTS_DIR", os.path.join(prj_dir, results_dir_name))
     return results_dir