From bbc7a8fdfcb67e5fa84c72a0ec5bb3ce910dea15 Mon Sep 17 00:00:00 2001 From: Konstantin Pavlov <1517853+kpavlov@users.noreply.github.com> Date: Thu, 11 Sep 2025 12:48:07 +0300 Subject: [PATCH] Refactor workflow, run on main (#251) * Refactor workflow - Rename workflow to 'Build' - Add push trigger on main branch - Cancel workflow only when the run is NOT on `main` branch - Update Gradle setup * fix CodeQL * Update CodeQL workflow and Gradle properties - Enable ignoring disabled targets for Kotlin Native. - Refine CodeQL build command to specify module targets and rerun tasks. * Fix auto-merge condition in workflow - Adjust failure condition to only disable auto-merge for pull requests. --- .github/workflows/{validate-pr.yml => build.yml} | 16 +++++++++++----- .github/workflows/codeql.yml | 8 +++++--- gradle.properties | 1 + 3 files changed, 17 insertions(+), 8 deletions(-) rename .github/workflows/{validate-pr.yml => build.yml} (67%) diff --git a/.github/workflows/validate-pr.yml b/.github/workflows/build.yml similarity index 67% rename from .github/workflows/validate-pr.yml rename to .github/workflows/build.yml index 56adb6c..fc4123e 100644 --- a/.github/workflows/validate-pr.yml +++ b/.github/workflows/build.yml @@ -1,13 +1,16 @@ -name: Validate PR +name: Build on: workflow_dispatch: pull_request: branches: [ main ] + push: + branches: [ main ] concurrency: - group: ${{ github.workflow }}-${{ github.ref }} - cancel-in-progress: true + group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }} + # Cancel only when the run is NOT on `main` branch + cancel-in-progress: ${{ github.ref != 'refs/heads/main' }} jobs: validate-pr: @@ -24,8 +27,11 @@ jobs: - name: Setup Gradle uses: gradle/actions/setup-gradle@v4 + with: + add-job-summary: 'always' + cache-read-only: true - - name: Clean Build with Gradle + - name: Build with Gradle run: ./gradlew clean build - name: Upload JUnit test results @@ -36,7 +42,7 @@ jobs: path: '**/build/test-results/test/*.xml' - name: Disable Auto-Merge on Fail - if: failure() + if: failure() && github.event_name == 'pull_request' run: gh pr merge --disable-auto "$PR_URL" env: PR_URL: ${{github.event.pull_request.html_url}} diff --git a/.github/workflows/codeql.yml b/.github/workflows/codeql.yml index 29ba0f5..63c798f 100644 --- a/.github/workflows/codeql.yml +++ b/.github/workflows/codeql.yml @@ -29,7 +29,9 @@ jobs: steps: - name: Checkout repository - uses: actions/checkout@v4 + uses: actions/checkout@v5 + with: + fetch-depth: 0 - uses: actions/setup-java@v5 with: @@ -52,14 +54,14 @@ jobs: - name: Build Kotlin sources run: | - ./gradlew \ + ./gradlew \ :kotlin-sdk-core:compileKotlinJvm \ :kotlin-sdk-client:compileKotlinJvm \ :kotlin-sdk-server:compileKotlinJvm \ :kotlin-sdk:compileKotlinJvm \ :kotlin-sdk-test:compileKotlinJvm \ -Pkotlin.incremental=false \ - --no-daemon --stacktrace + --no-daemon --stacktrace --rerun-tasks - name: Analyze uses: github/codeql-action/analyze@v3 diff --git a/gradle.properties b/gradle.properties index a868081..85b9566 100644 --- a/gradle.properties +++ b/gradle.properties @@ -9,3 +9,4 @@ kotlin.code.style=official kotlin.daemon.jvmargs=-Xmx4G # MPP kotlin.mpp.enableCInteropCommonization=true +kotlin.native.ignoreDisabledTargets=true