Merge branch 'migrate_to_settings'

danger · May 25, 2018 · 01a0357 · 01a0357
2 parents 6cc0eb5 + 0ad277d
commit 01a0357
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 3 deletions.
diff --git a/Dockerfile.runner b/Dockerfile.runner
@@ -4,10 +4,8 @@ ADD . /app
 WORKDIR /app
 
 ARG SOURCE_COMMIT
-RUN test -n "$SOURCE_COMMIT"
 ENV SOURCE_COMMIT $SOURCE_COMMIT
 
-
 # This will also trigger the build process
 RUN yarn install
 

diff --git a/source/db/getDB.ts b/source/db/getDB.ts
@@ -10,6 +10,7 @@ const hasJSONDef = !!process.env.DATABASE_JSON_FILE
 const hasPerilAPIURL = !!process.env.PUBLIC_API_ROOT_URL
 const hasHyperEnv = !!process.env.x_hyper_content_sha256
 
+/** There are three runtime environments for Peril, this says which one it is */
 export const runtimeEnvironment = hasJSONDef
   ? RuntimeEnvironment.Standalone
   : hasPerilAPIURL

diff --git a/source/runner/triggerSandboxRun.ts b/source/runner/triggerSandboxRun.ts
@@ -65,7 +65,6 @@ export const triggerSandboxDangerRun = async (
 
   // Grab the installation env vars
   const envVars = await getEnvVars(installation.iID)
-  console.log("OK", envVars)
   const stdOUT: PerilRunnerBootstrapJSON = {
     installation,
     payload,