Once the clone and the checkout succeeded, get rid of the bare repository

We start out with a bare, shallow clone of `git-sdk-32`/`git-sdk-64`.
This lives in the current directory, as `.tmp/`. Some GitHub workflows
are very picky about the current directory and do not want any untracked
files flying about.

Most notably, Git's own test suite wants to make sure that there are no
unexpected files in the work tree after a successful build, and `.tmp`
counts for "unexpected" here.

Let's remove the bare repository once we no longer need it.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>

Author: dscho

.github/workflows/test.yml

@@ -42,7 +42,8 @@ jobs:
       - name: Verify that the Bash of Git for Windows' SDK is used
         shell: bash
         run: |
-          set -x
+          set -ex
+          test ! -e .tmp
           echo "This is the MSYS2 pseudo root: $(cygpath -aw /)"
           test "gcc is /mingw64/bin/gcc" = "$(type gcc)"
           test "prove is /usr/bin/core_perl/prove" = "$(type prove)"

src/git.ts

@@ -2,6 +2,7 @@ import * as core from '@actions/core'
 import {ChildProcess, spawn} from 'child_process'
 import {Octokit} from '@octokit/rest'
 import {delimiter} from 'path'
+import * as fs from 'fs'
 
 const gitForWindowsUsrBinPath = 'C:/Program Files/Git/usr/bin'
 const gitExePath = 'C:/Program Files/Git/cmd/git.exe'
@@ -160,7 +161,7 @@ export async function getViaGit(
         child.on('close', code => {
           core.endGroup()
           if (code === 0) {
-            resolve()
+            fs.rm('.tmp', {recursive: true}, () => resolve())
           } else {
             reject(new Error(`process exited with code ${code}`))
           }