feat: only create deployment for pull requests (#23)

Also skip setting it to IN_PROGRESS -- just create it and set it to
success.

Author: stayradiated

bin/index.js

@@ -102,18 +102,6 @@ const main = async () => {
     }
 
     if (deployingMainApp) {
-      const deploymentId = await createGithubDeployment({
-        githubAPIKey,
-        ref: target.commitSHA,
-      })
-      await setGithubDeploymentStatus({
-        githubAPIKey,
-        ref: target.commitSHA,
-        deploymentId,
-        state: GithubDeploymentState.IN_PROGRESS,
-        environmentUrl: target.mainAppUrl,
-      })
-
       await heroku.releaseContainer({
         appName: target.mainAppName,
         processTypes: ['web', 'worker'],
@@ -145,13 +133,19 @@ const main = async () => {
         })
       }
 
-      await setGithubDeploymentStatus({
-        githubAPIKey,
-        ref: target.commitSHA,
-        deploymentId,
-        state: GithubDeploymentState.SUCCESS,
-        environmentUrl: target.mainAppUrl,
-      })
+      if (target.hasPullRequest) {
+        const deploymentId = await createGithubDeployment({
+          githubAPIKey,
+          ref: target.commitSHA,
+        })
+        await setGithubDeploymentStatus({
+          githubAPIKey,
+          ref: target.commitSHA,
+          deploymentId,
+          state: GithubDeploymentState.SUCCESS,
+          environmentUrl: target.mainAppUrl,
+        })
+      }
     }
   }
 }

src/index.ts

@@ -9,6 +9,7 @@ type Target = {
   pipelineName: string,
   pipelineStage: string,
   createAppIfNotExists: boolean,
+  hasPullRequest: boolean,
 }
 
 const getDeploymentTargets = async (): Promise<Target[]> => {
@@ -31,6 +32,7 @@ const getDeploymentTargets = async (): Promise<Target[]> => {
               pipelineName,
               pipelineStage: 'staging',
               createAppIfNotExists: false,
+              hasPullRequest: false,
             },
           ]
         }
@@ -45,6 +47,7 @@ const getDeploymentTargets = async (): Promise<Target[]> => {
               pipelineName,
               pipelineStage: 'staging',
               createAppIfNotExists: false,
+              hasPullRequest: false,
             },
           ]
         }
@@ -59,6 +62,7 @@ const getDeploymentTargets = async (): Promise<Target[]> => {
               pipelineName,
               pipelineStage: 'production',
               createAppIfNotExists: false,
+              hasPullRequest: false,
             },
             {
               commitSHA,
@@ -69,6 +73,7 @@ const getDeploymentTargets = async (): Promise<Target[]> => {
               pipelineName,
               pipelineStage: 'production',
               createAppIfNotExists: false,
+              hasPullRequest: false,
             },
           ]
         }
@@ -96,6 +101,7 @@ const getDeploymentTargets = async (): Promise<Target[]> => {
           pipelineName,
           pipelineStage: 'development',
           createAppIfNotExists: true,
+          hasPullRequest: true,
         },
       ]
     }