From bd9f883bb63937b78824d0bceea52570648fc51b Mon Sep 17 00:00:00 2001
From: Harsh Shandilya <me@msfjarvis.dev>
Date: Fri, 10 Jun 2022 23:27:56 +0530
Subject: Improve Kotlin build caching (#1947)

* Update Gradle properties with Kotlin 1.7 build things

* Disable build tooling metadata task

* Revert "Revert "github: stop disabling configuration cache in CI""

This reverts commit 08951643f51a5a62ca2419cb3a5e935b1d7de413.

* Store build reports as artifacts
---
 .github/workflows/pull_request.yml | 54 +++++++++++++++++++++++++++++++++-----
 1 file changed, 48 insertions(+), 6 deletions(-)

(limited to '.github/workflows/pull_request.yml')

diff --git a/.github/workflows/pull_request.yml b/.github/workflows/pull_request.yml
index 41f4ed76..c9a0d166 100644
--- a/.github/workflows/pull_request.yml
+++ b/.github/workflows/pull_request.yml
@@ -42,7 +42,14 @@ jobs:
       if: "${{ steps.service-changed.outputs.result == 'true' }}"
       uses: gradle/gradle-build-action@v2.2.0
       with:
-        arguments: --no-configuration-cache spotlessCheck
+        arguments: spotlessCheck
+
+    - name: Upload Kotlin build report
+      if: "${{ always() }}"
+      uses: actions/upload-artifact@v3.1.0
+      with:
+          name: Build report
+          path: build/reports/kotlin-build/
 
   unit-tests:
     runs-on: ubuntu-latest
@@ -76,7 +83,7 @@ jobs:
       if: "${{ steps.service-changed.outputs.result == 'true' }}"
       uses: gradle/gradle-build-action@v2.2.0
       with:
-        arguments: --no-configuration-cache test -PslimTests
+        arguments: test -PslimTests
 
     - name: (Fail-only) Upload test report
       if: "${{ failure() }}"
@@ -85,6 +92,13 @@ jobs:
           name: Test report
           path: app/build/reports
 
+    - name: Upload Kotlin build report
+      if: "${{ always() }}"
+      uses: actions/upload-artifact@v3.1.0
+      with:
+          name: Build report
+          path: build/reports/kotlin-build/
+
   build-apks:
     runs-on: ubuntu-latest
     steps:
@@ -117,7 +131,14 @@ jobs:
       if: "${{ steps.service-changed.outputs.result == 'true' }}"
       uses: gradle/gradle-build-action@v2.2.0
       with:
-        arguments: --no-configuration-cache assembleFreeDebug assembleNonFreeDebug
+        arguments: assembleFreeDebug assembleNonFreeDebug
+
+    - name: Upload Kotlin build report
+      if: "${{ always() }}"
+      uses: actions/upload-artifact@v3.1.0
+      with:
+          name: Build report
+          path: build/reports/kotlin-build/
 
   check-api:
     runs-on: ubuntu-latest
@@ -151,7 +172,14 @@ jobs:
       if: "${{ steps.service-changed.outputs.result == 'true' }}"
       uses: gradle/gradle-build-action@v2.2.0
       with:
-        arguments: --no-configuration-cache apiCheck
+        arguments: apiCheck
+
+    - name: Upload Kotlin build report
+      if: "${{ always() }}"
+      uses: actions/upload-artifact@v3.1.0
+      with:
+          name: Build report
+          path: build/reports/kotlin-build/
 
   lint-debug:
     runs-on: ubuntu-latest
@@ -185,7 +213,14 @@ jobs:
       if: "${{ steps.service-changed.outputs.result == 'true' }}"
       uses: gradle/gradle-build-action@v2.2.0
       with:
-        arguments: --no-configuration-cache lintDebug
+        arguments: lintDebug
+
+    - name: Upload Kotlin build report
+      if: "${{ always() }}"
+      uses: actions/upload-artifact@v3.1.0
+      with:
+          name: Build report
+          path: build/reports/kotlin-build/
 
   assemble-release-apks:
     runs-on: ubuntu-latest
@@ -219,4 +254,11 @@ jobs:
       if: "${{ steps.service-changed.outputs.result == 'true' }}"
       uses: gradle/gradle-build-action@v2.2.0
       with:
-        arguments: --no-configuration-cache aNFR
+        arguments: aNFR
+
+    - name: Upload Kotlin build report
+      if: "${{ always() }}"
+      uses: actions/upload-artifact@v3.1.0
+      with:
+          name: Build report
+          path: build/reports/kotlin-build/
-- 
cgit v1.2.3