GHA: test site build (#60)

* test the site still builds on PRs, and cache NPM, and minor Gradle cleanups

* fix cache npm key

Author: aSemy

.github/workflows/deploy-site.yml

@@ -31,6 +31,19 @@ jobs:
         with:
           gradle-home-cache-cleanup: true
 
+      - name: Cache NPM
+        uses: actions/cache@v3
+        env:
+          cache-name: cache-npm
+        with:
+          key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/package-lock.json') }}
+          path: |
+            ./site/node_modules
+          restore-keys: |
+            ${{ runner.os }}-build-${{ env.cache-name }}-
+            ${{ runner.os }}-build-
+            ${{ runner.os }}-
+
       - name: docusaurus build
         run: ./gradlew docusaurusBuild
 

.github/workflows/gradle_task.yml

@@ -35,27 +35,40 @@ jobs:
     name: "./gradlew ${{ inputs.gradle-task}} @ ${{ inputs.runs-on }}"
     timeout-minutes: 60
     steps:
-      -  name: Checkout the repo
-         uses: actions/checkout@v3
-
-      -  name: Setup JDK
-         uses: actions/setup-java@v3
-         with:
-           distribution: temurin
-           java-version: 11
-
-      -  uses: gradle/gradle-build-action@v2
-         with:
-           gradle-home-cache-cleanup: true
-
-      -  name: Run ${{ inputs.gradle-task }}
-         run: >-
-           ./gradlew ${{ inputs.gradle-task }}
-
-      -  name: Upload build reports
-         if: failure()
-         uses: actions/upload-artifact@v3
-         with:
-           name: build-report-${{ runner.os }}-${{ github.action }}
-           path: "**/build/reports/"
-           if-no-files-found: ignore
+      - name: Checkout the repo
+        uses: actions/checkout@v3
+
+      - name: Setup JDK
+        uses: actions/setup-java@v3
+        with:
+          distribution: temurin
+          java-version: 11
+
+      - uses: gradle/gradle-build-action@v2
+        with:
+          gradle-home-cache-cleanup: true
+
+      - name: Cache NPM
+        uses: actions/cache@v3
+        env:
+          cache-name: cache-npm
+        with:
+          key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/package-lock.json') }}
+          path: |
+            ./site/node_modules
+          restore-keys: |
+            ${{ runner.os }}-build-${{ env.cache-name }}-
+            ${{ runner.os }}-build-
+            ${{ runner.os }}-
+
+      - name: Run ${{ inputs.gradle-task }}
+        run: >-
+          ./gradlew ${{ inputs.gradle-task }}
+
+      - name: Upload build reports
+        if: failure()
+        uses: actions/upload-artifact@v3
+        with:
+          name: build-report-${{ runner.os }}-${{ github.action }}
+          path: "**/build/reports/"
+          if-no-files-found: ignore

.github/workflows/tests.yml

@@ -29,3 +29,9 @@ jobs:
       runs-on: ${{ matrix.os }}
       gradle-task: >-
         check -Pkxstsgen_enableTsCompileTests=true --stacktrace
+
+  build-site:
+    uses: ./.github/workflows/gradle_task.yml
+    with:
+      runs-on: ubuntu-latest
+      gradle-task: docusaurusBuild

buildSrc/src/main/kotlin/buildsrc/convention/kotlin-mpp.gradle.kts

@@ -2,7 +2,6 @@ package buildsrc.convention
 
 import buildsrc.config.relocateKotlinJsStore
 import org.jetbrains.kotlin.gradle.dsl.KotlinMultiplatformExtension
-import org.jetbrains.kotlin.gradle.plugin.mpp.KotlinJvmCompilation
 import org.jetbrains.kotlin.gradle.targets.jvm.KotlinJvmTarget
 
 
@@ -23,8 +22,6 @@ extensions.configure<KotlinMultiplatformExtension> {
         languageVersion = "1.7"
         apiVersion = "1.7"
       }
-      if (this is KotlinJvmCompilation) {
-      }
     }
   }
 

settings.gradle.kts

@@ -12,6 +12,7 @@ include(
 )
 
 enableFeaturePreview("TYPESAFE_PROJECT_ACCESSORS")
+enableFeaturePreview("STABLE_CONFIGURATION_CACHE")
 
 dependencyResolutionManagement {
   @Suppress("UnstableApiUsage") // Central declaration of repositories is an incubating feature