clean up some comments

Author: etpinard

circle.yml

@@ -1,17 +1,17 @@
 general:
   artifacts:
-      - "build/test_images/" # relative to the build directory
-      - "build/test_images_diff/" # relative to the build directory
+      - "build/test_images/"
+      - "build/test_images_diff/"
 
 machine:
   node:
-    version: 4.2.1 
+    version: 4.2.1
   services:
     - docker
-    
+
 
 dependencies:
-  pre: 
+  pre:
     - docker pull plotly/imageserver:latest
   post:
     - npm run cibuild

tasks/baseline.sh

@@ -2,23 +2,12 @@
 #
 # TODO adapt this for Windows
 #
-# TODO add package.json config arguments to configure:
-#       - container name,
-#       - ports and
-#       - imageserver version
-#
 # ===============================================================================
 
-CONTAINER_NAME="imagetest"
-IMAGE_NAME="registry.plot.ly:5000/imageserver"
-IMAGE_VERSION="1.3.0"
+CONTAINER_NAME="imagetest"  # same as in docker-compose.yml
 
-# Run docker container:
-#
-# docker run -d --name $CONTAINER_NAME \
-#     -v $PWD/plotly.js:/var/www/streambed/image_server/plotly.js \
-#     -p 9010:9010 -p 2022:22 \
-#     $IMAGE_NAME:[$IMAGE_VERSION]
+# create/run/start docker container with:
+#   $ docker-compose up -d
 
 CMD=(
     "cd /var/www/streambed/image_server/plotly.js &&"

tasks/test_image.sh

@@ -2,23 +2,12 @@
 #
 # TODO adapt this for Windows
 #
-# TODO add package.json config arguments to configure:
-#       - container name,
-#       - ports and
-#       - imageserver version
-#
 # ===============================================================================
 
-CONTAINER_NAME="imagetest"
-IMAGE_NAME="registry.plot.ly:5000/imageserver"
-IMAGE_VERSION="1.3.0"
+CONTAINER_NAME="imagetest"  # same as in docker-compose.yml
 
-# Run docker container:
-#
-# docker run -d --name $CONTAINER_NAME \
-#     -v $PWD/plotly.js:/var/www/streambed/image_server/plotly.js \
-#     -p 9010:9010 -p 2022:22 \
-#     $IMAGE_NAME:[$IMAGE_VERSION]
+# create/run/start docker container with:
+#   $ docker-compose up -d
 
 CMD=(
     "cd /var/www/streambed/image_server/plotly.js &&"

test/image/README.md

@@ -71,6 +71,9 @@ docker images
 docker ps -a
 ```
 
+whereas `docker ps` lists only the started containers.
+
+
 ### Stop your testing container
 
 Inside your `plotly.js` directory, run
@@ -87,4 +90,4 @@ Inside your `plotly.js` directory, run
 docker-compose rm -f
 ```
 
-For more comprehensive information about docker, please refer to [docker document](http://docs.docker.com/)
+For more comprehensive information about docker, please refer to the [docker docs](http://docs.docker.com/).