From 27ef7c102d44adda8409eb138e372d0bc7ce50f1 Mon Sep 17 00:00:00 2001 From: Giancarlo Buenaflor Date: Fri, 4 Apr 2025 01:20:44 +0200 Subject: [PATCH 1/3] Try building only the SPM sample --- .github/workflows/kotlin-multiplatform.yml | 168 ++++++++++----------- 1 file changed, 84 insertions(+), 84 deletions(-) diff --git a/.github/workflows/kotlin-multiplatform.yml b/.github/workflows/kotlin-multiplatform.yml index 5605eb24..729f0159 100644 --- a/.github/workflows/kotlin-multiplatform.yml +++ b/.github/workflows/kotlin-multiplatform.yml @@ -11,89 +11,89 @@ concurrency: cancel-in-progress: true jobs: - build-apple: - runs-on: macos-latest-xlarge - - steps: - - uses: actions/checkout@v4 - - - name: JDK setup - uses: actions/setup-java@v3 - with: - java-version: 17 - distribution: temurin - - - name: Cached Konan - uses: actions/cache@v4 - with: - path: ~/.konan - key: ${{ runner.os }}-konan-${{ hashFiles('**/*.gradle*') }}-${{ hashFiles('**/gradle/wrapper/gradle-wrapper.properties') }} - restore-keys: ${{ runner.os }}-konan- - - - name: Cached Gradle - uses: gradle/gradle-build-action@842c587ad8aa4c68eeba24c396e15af4c2e9f30a - - - name: Apple build - run: | - ./scripts/build-apple.sh sentry-kotlin-multiplatform - env: - SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }} - - build-apple-samples: - runs-on: macos-latest-xlarge - - steps: - - uses: actions/checkout@v4 - - - name: JDK setup - uses: actions/setup-java@v3 - with: - java-version: 17 - distribution: temurin - - - name: Cached Konan - uses: actions/cache@v4 - with: - path: ~/.konan - key: ${{ runner.os }}-konan-${{ hashFiles('**/*.gradle*') }}-${{ hashFiles('**/gradle/wrapper/gradle-wrapper.properties') }} - restore-keys: ${{ runner.os }}-konan- - - - name: Cached Gradle - uses: gradle/gradle-build-action@842c587ad8aa4c68eeba24c396e15af4c2e9f30a - - - name: Apple build - run: | - make buildAppleSamples - - build-jvm: - runs-on: ubuntu-latest - - steps: - - uses: actions/checkout@v4 - - - name: JDK setup - uses: actions/setup-java@v3 - with: - java-version: 17 - distribution: temurin - - - name: Cached Gradle - uses: gradle/gradle-build-action@842c587ad8aa4c68eeba24c396e15af4c2e9f30a - - - name: JVM build - run: | - ./scripts/build-jvm.sh sentry-kotlin-multiplatform - make createCoverageReports - env: - SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }} - GRADLE_OPTS: -Dorg.gradle.jvmargs="-Xmx3g" - - # Kover coverage currently only works for JVM - - name: Upload coverage to Codecov - uses: codecov/codecov-action@5ecb98a3c6b747ed38dc09f787459979aebb39be # pin@v4 - with: - name: sentry-kotlin-multiplatform - token: ${{ secrets.CODECOV_TOKEN }} +# build-apple: +# runs-on: macos-latest-xlarge +# +# steps: +# - uses: actions/checkout@v4 +# +# - name: JDK setup +# uses: actions/setup-java@v3 +# with: +# java-version: 17 +# distribution: temurin +# +# - name: Cached Konan +# uses: actions/cache@v4 +# with: +# path: ~/.konan +# key: ${{ runner.os }}-konan-${{ hashFiles('**/*.gradle*') }}-${{ hashFiles('**/gradle/wrapper/gradle-wrapper.properties') }} +# restore-keys: ${{ runner.os }}-konan- +# +# - name: Cached Gradle +# uses: gradle/gradle-build-action@842c587ad8aa4c68eeba24c396e15af4c2e9f30a +# +# - name: Apple build +# run: | +# ./scripts/build-apple.sh sentry-kotlin-multiplatform +# env: +# SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }} +# +# build-apple-samples: +# runs-on: macos-latest-xlarge +# +# steps: +# - uses: actions/checkout@v4 +# +# - name: JDK setup +# uses: actions/setup-java@v3 +# with: +# java-version: 17 +# distribution: temurin +# +# - name: Cached Konan +# uses: actions/cache@v4 +# with: +# path: ~/.konan +# key: ${{ runner.os }}-konan-${{ hashFiles('**/*.gradle*') }}-${{ hashFiles('**/gradle/wrapper/gradle-wrapper.properties') }} +# restore-keys: ${{ runner.os }}-konan- +# +# - name: Cached Gradle +# uses: gradle/gradle-build-action@842c587ad8aa4c68eeba24c396e15af4c2e9f30a +# +# - name: Apple build +# run: | +# make buildAppleSamples +# +# build-jvm: +# runs-on: ubuntu-latest +# +# steps: +# - uses: actions/checkout@v4 +# +# - name: JDK setup +# uses: actions/setup-java@v3 +# with: +# java-version: 17 +# distribution: temurin +# +# - name: Cached Gradle +# uses: gradle/gradle-build-action@842c587ad8aa4c68eeba24c396e15af4c2e9f30a +# +# - name: JVM build +# run: | +# ./scripts/build-jvm.sh sentry-kotlin-multiplatform +# make createCoverageReports +# env: +# SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }} +# GRADLE_OPTS: -Dorg.gradle.jvmargs="-Xmx3g" +# +# # Kover coverage currently only works for JVM +# - name: Upload coverage to Codecov +# uses: codecov/codecov-action@5ecb98a3c6b747ed38dc09f787459979aebb39be # pin@v4 +# with: +# name: sentry-kotlin-multiplatform +# token: ${{ secrets.CODECOV_TOKEN }} test-samples: runs-on: macos-latest-xlarge @@ -112,4 +112,4 @@ jobs: - name: Test samples run: | - ./gradlew allTests -p sentry-samples + ./gradlew allTests -p sentry-samples/kmp-app-spm From 0bad9b90836e101032e468214e883ed57cfffc5a Mon Sep 17 00:00:00 2001 From: Giancarlo Buenaflor Date: Fri, 4 Apr 2025 01:25:35 +0200 Subject: [PATCH 2/3] dont use cached gradle buidl --- .github/workflows/kotlin-multiplatform.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/kotlin-multiplatform.yml b/.github/workflows/kotlin-multiplatform.yml index 729f0159..d925cde5 100644 --- a/.github/workflows/kotlin-multiplatform.yml +++ b/.github/workflows/kotlin-multiplatform.yml @@ -107,9 +107,9 @@ jobs: java-version: 17 distribution: temurin - - name: Cached Gradle - uses: gradle/gradle-build-action@842c587ad8aa4c68eeba24c396e15af4c2e9f30a - +# - name: Cached Gradle +# uses: gradle/gradle-build-action@842c587ad8aa4c68eeba24c396e15af4c2e9f30a +# - name: Test samples run: | ./gradlew allTests -p sentry-samples/kmp-app-spm From 3e4e2594477a1f495225efc4a61b06698c62dcfa Mon Sep 17 00:00:00 2001 From: Giancarlo Buenaflor Date: Fri, 4 Apr 2025 01:42:50 +0200 Subject: [PATCH 3/3] update --- .github/workflows/kotlin-multiplatform.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/.github/workflows/kotlin-multiplatform.yml b/.github/workflows/kotlin-multiplatform.yml index d925cde5..1ff95e47 100644 --- a/.github/workflows/kotlin-multiplatform.yml +++ b/.github/workflows/kotlin-multiplatform.yml @@ -110,6 +110,10 @@ jobs: # - name: Cached Gradle # uses: gradle/gradle-build-action@842c587ad8aa4c68eeba24c396e15af4c2e9f30a # + # temporary for testing + - name: Remove DerivedData first + run: rm -rf ~/Library/Developer/Xcode/DerivedData + - name: Test samples run: | ./gradlew allTests -p sentry-samples/kmp-app-spm