add notes for future me

LDMX-Software · Mar 15, 2024 · 698cafa · 698cafa
1 parent 21670ea
commit 698cafa
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/scripts/README.md b/scripts/README.md
@@ -69,7 +69,7 @@ scripts wrapping denv for a specific shell should do the following.
 - Define the `ldmx` function with CLI defined below
 
 ### `ldmx` CLI Specification
-v0.0.0
+v0.0.0: just an idea
 
 - `ldmx`
   - `help` : print help message describing the CLI

diff --git a/scripts/ldmx-denv-init.sh b/scripts/ldmx-denv-init.sh
@@ -117,6 +117,8 @@ export LDMX_BASE
 
 ###############################################################################
 # Check if the LDMX denv is initialized. If not, do a default initialization.
+# TODO: update to `denv check --workspace` introduced in denv v0.7.0 to avoid
+#       reliance on internal implementation on how denv stores its config
 ###############################################################################
 if [ ! -f "${LDMX_BASE}/.denv/config" ]; then
   denv init --clean-env --name "ldmx" "ldmx/dev:latest" "${LDMX_BASE}"