fix ci scripts: check for docker -t

insitro · Sep 7, 2023 · d6b62d8 · d6b62d8
1 parent 905fd27
commit d6b62d8
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/.run.docker.sh b/.run.docker.sh
@@ -4,12 +4,19 @@ export DOCKER_IMG=redun_python_test:redun_python_test
 export DOCKER_CONTAINER_NAME=redun_python_test
 
 docker build . --tag $DOCKER_IMG >/dev/null
+docker-compose up -d
 
 ( docker rm -f $DOCKER_CONTAINER_NAME || true ) 2>&1 >/dev/null
 
+if [ -t 0 ]; then
+        TERM="-i -t"
+else
+        TERM="-i "
+fi
+
 docker run \
          -u "$(id -u):$(id -g)" \
-        -it \
+        $TERM \
         --rm \
         --name $DOCKER_CONTAINER_NAME \
         -e PGUSER=postgres -e PGPASSWORD=postgres \

diff --git a/docs/source/developing.md b/docs/source/developing.md
@@ -9,7 +9,7 @@ tocpdeth: 3
 Run `./pytest.sh` with the arguments that you would pass to `pytest`, for example:
 
 ```
-./pytest.sh -k static
+./pytest.sh -k static --pdb
 ```
 
 The lints also have their own shortcut: `./lint.sh`