Merge pull request #503 from rtyley/do-not-persist-summary-that-is-sp…

…ecific-a-single-run Don't persist the `run-summary.md` file beyond its own run
scala-steward-org · Jun 16, 2023 · 916e71f · 916e71f
2 parents b6f8161 + 2a1bf31
commit 916e71f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/src/modules/workspace.test.ts b/src/modules/workspace.test.ts
@@ -182,6 +182,7 @@ test('`Workspace.saveWorkspaceCache()` → saves cache', async t => {
   const expected: string[] = [
     'rmRF("/home/scala-steward/workspace/store/refresh_error")',
     'rmRF("/home/scala-steward/workspace/repos")',
+    'rmRF("/home/scala-steward/workspace/run-summary.md")',
     'readFileSync("/home/scala-steward/repos.md")',
     `saveCache([/home/scala-steward/workspace], "scala-steward-acc000fd-${now}")`,
   ]

diff --git a/src/modules/workspace.ts b/src/modules/workspace.ts
@@ -73,6 +73,9 @@ export class Workspace {
       await this.files.rmRF(path.join(this.workspace.value, 'store', 'refresh_error'))
       await this.files.rmRF(path.join(this.workspace.value, 'repos'))
 
+      // Don't persist a summary that's specific to this run
+      await this.files.rmRF(this.runSummary_md)
+
       const hash = this.hashFile(this.repos_md.value)
 
       await this.cache.saveCache([this.workspace.value], `scala-steward-${hash}-${Date.now().toString()}`)