fix: ensure git found on trop runner

Aptfile

@@ -0,0 +1,2 @@
+# Heroku-24 stack doesn't include git by default.
+git-all

app.json

@@ -11,15 +11,15 @@
       "required": true
     }
   },
-  "formation": {
-  },
-  "addons": [
-
-  ],
+  "formation": {},
+  "addons": [],
   "buildpacks": [
+    {
+      "url": "heroku-community/apt"
+    },
     {
       "url": "heroku/nodejs"
     }
   ],
-  "stack": "heroku-22"
+  "stack": "heroku-24"
 }

src/index.ts

@@ -28,12 +28,7 @@ import {
   updateBackportValidityCheck,
 } from './utils/checks-util';
 import { register } from './utils/prom';
-import {
-  SimpleWebHookRepoContext,
-  WebHookIssueContext,
-  WebHookPR,
-  WebHookPRContext,
-} from './types';
+import { SimpleWebHookRepoContext, WebHookPR, WebHookPRContext } from './types';
 
 // Built in fetch doesn't support global-agent...
 // @ts-ignore
@@ -423,7 +418,7 @@ const probotHandler: ApplicationFunction = async (robot, { getRouter }) => {
   );
 
   // Backport pull requests to labeled targets when PR is merged.
-  robot.on('pull_request.closed', async (context: WebHookPRContext) => {
+  robot.on('pull_request.closed', async (context) => {
     const { pull_request: pr } = context.payload;
 
     const oldPRNumbers = getPRNumbersFromPRBody(pr, true);
@@ -471,7 +466,7 @@ const probotHandler: ApplicationFunction = async (robot, { getRouter }) => {
   const TROP_COMMAND_PREFIX = '/trop ';
 
   // Manually trigger backporting process on trigger comment phrase.
-  robot.on('issue_comment.created', async (context: WebHookIssueContext) => {
+  robot.on('issue_comment.created', async (context) => {
     const { issue, comment } = context.payload;
 
     const isPullRequest = (i: { number: number; html_url: string }) =>

src/utils.ts

@@ -1,5 +1,6 @@
 import fetch from 'node-fetch';
 import * as fs from 'fs-extra';
+import { execSync } from 'child_process';
 import Queue from 'queue';
 import simpleGit from 'simple-git';
 
@@ -29,7 +30,7 @@ import {
   WebHookPR,
   WebHookRepoContext,
 } from './types';
-import { Context, Probot } from 'probot';
+import { Probot } from 'probot';
 
 const { parse: parseDiff } = require('what-the-diff');
 
@@ -475,6 +476,17 @@ export const backportImpl = async (
     }
   }
 
+  const gitExists = execSync('which git').toString().trim();
+  if (/git not found/.test(gitExists)) {
+    await context.octokit.issues.createComment(
+      context.repo({
+        body: `Git not found - unable to proceed with backporting to ${targetBranch}`,
+        issue_number: pr.number,
+      }),
+    );
+    return;
+  }
+
   const base = pr.base;
   const slug = `${base.repo.owner.login}/${base.repo.name}`;
   const bp = `backport from PR #${pr.number} to "${targetBranch}"`;