Improve logging

gradle · Aug 1, 2024 · e0d2343 · e0d2343
1 parent 45b944d
commit e0d2343
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 11 deletions.
diff --git a/sources/src/caching/cache-cleaner.ts b/sources/src/caching/cache-cleaner.ts
@@ -28,8 +28,6 @@ export class CacheCleaner {
 
     // Visible for testing
     async forceCleanupFilesOlderThan(cleanTimestamp: string): Promise<void> {
-        core.info(`Cleaning up caches before ${cleanTimestamp}`)
-
         // Run a dummy Gradle build to trigger cache cleanup
         const cleanupProjectDir = path.resolve(this.tmpDir, 'dummy-cleanup-project')
         fs.mkdirSync(cleanupProjectDir, {recursive: true})
@@ -59,7 +57,7 @@ export class CacheCleaner {
 
         const executable = await provisioner.provisionGradle('current')
 
-        await core.group('Executing Gradle build to clean up caches', async () => {
+        await core.group(`Executing Gradle to clean up caches before ${cleanTimestamp}`, async () => {
             await this.executeCleanupBuild(executable!, cleanupProjectDir)
         })
     }

diff --git a/sources/src/caching/caches.ts b/sources/src/caching/caches.ts
@@ -115,12 +115,10 @@ export async function save(
 }
 
 async function performCacheCleanup(gradleUserHome: string): Promise<void> {
-    await core.group('Performing cache cleanup', async () => {
-        const cacheCleaner = new CacheCleaner(gradleUserHome, process.env['RUNNER_TEMP']!)
-        try {
-            await cacheCleaner.forceCleanup()
-        } catch (e) {
-            core.warning(`Cache cleanup failed. Will continue. ${String(e)}`)
-        }
-    })
+    const cacheCleaner = new CacheCleaner(gradleUserHome, process.env['RUNNER_TEMP']!)
+    try {
+        await cacheCleaner.forceCleanup()
+    } catch (e) {
+        core.warning(`Cache cleanup failed. Will continue. ${String(e)}`)
+    }
 }