Merge pull request #589 from plotly/improve-circleci

Propose various improvement to circleci workflow

Former-commit-id: 2f8ed51

Author: Xing Han Lu

.circleci/config.yml

@@ -9,17 +9,13 @@ jobs:
     steps:
       - checkout
       - run:
-          name: install black and flake8
+          name: install black
           command: |
-            sudo pip install black==19.10b0 flake8
+            sudo pip install black==19.10b0
       - run:
           name: run black
           command: |
             black . --check
-      - run:
-          name: run flake8
-          command: |
-            flake8 --select=E265,E711,W291
   deploy_to_playground:
     docker:
       - image: circleci/python:3.6-stretch
@@ -106,7 +102,7 @@ jobs:
 
             for APP in $APPS_MODIFIED
             do
-              DASH_ENTERPRISE_URL="$DASH_GALLERY_ENTERPRISE_URL" DASH_ENTERPRISE_API_KEY="$DASH_GALLERY_ENTERPRISE_API_KEY" PATH="$HOME/bin:$PATH" ./deploy "$APP"
+              CREATE_APP=true DASH_ENTERPRISE_URL="$DASH_GALLERY_ENTERPRISE_URL" DASH_ENTERPRISE_API_KEY="$DASH_GALLERY_ENTERPRISE_API_KEY" PATH="$HOME/bin:$PATH" ./deploy "$APP"
             done
 
 workflows:

deploy

@@ -94,7 +94,7 @@ main() {
   fi
 
   if [[ ! -d "$app_dir/assets" ]]; then
-    log-fail "App $APP is missing the assets directory"
+    log-warn "App $APP is missing the assets directory. Make sure to create one if you use asset files."
   fi
 
   log-header "Deploying $APP"
@@ -178,7 +178,7 @@ main() {
     log-info "Python app detected, injecting common python-specific files"
     cp -p -n "$SOURCE_DIR/app.json" "$tmp_full_clone/app.json"
     cp -p "$SOURCE_DIR/predeploy.py" "$tmp_full_clone/predeploy.py"
-    test -f "$tmp_full_clone/runtime.txt" || cp -p "$SOURCE_DIR/runtime.txt" "$tmp_full_clone/runtime.txt"
+    test -f "$tmp_full_clone/runtime.txt" || test -f "$SOURCE_DIR/runtime.txt" || cp -p "$SOURCE_DIR/runtime.txt" "$tmp_full_clone/runtime.txt"
   fi
 
   set +e