Compilation with build tools API #876

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

zwiora merged 1 commit into kotlin-community/dev from Q3/compilation-with-BTA

Oct 20, 2025

Collaborator

zwiora commented Sep 29, 2025

No description provided.

zwiora requested a review from dkrasnoff as a code owner

September 29, 2025 08:23

dkrasnoff force-pushed the kotlin-community/dev branch from 8fd7307 to 4f2f724 Compare

October 1, 2025 10:20

dkrasnoff reviewed

View reviewed changes

build.gradle.kts Outdated Show resolved Hide resolved

Collaborator

dkrasnoff commented Oct 6, 2025

It seems, that you've merged your changes with dkrasnov/KTL-724-provide-the-opportunity-to-pass-kotlin-compiler-arguments branch a little bit incorrectly.
I expect that there will be no indexation module in your branch.

dkrasnoff reviewed

View reviewed changes

common/build.gradle.kts Outdated Show resolved Hide resolved

indexation/build.gradle.kts Outdated Show resolved Hide resolved

dkrasnoff reviewed

View reviewed changes

src/main/kotlin/com/compiler/server/compiler/components/CompilationLogger.kt Show resolved Hide resolved

dkrasnoff reviewed

View reviewed changes

src/main/kotlin/com/compiler/server/compiler/components/CompletionProvider.kt Outdated Show resolved Hide resolved

dkrasnoff reviewed

View reviewed changes

src/main/kotlin/com/compiler/server/compiler/components/ErrorAnalyzer.kt Outdated Show resolved Hide resolved

dkrasnoff reviewed

View reviewed changes

src/main/kotlin/com/compiler/server/compiler/components/KotlinCompiler.kt Outdated Show resolved Hide resolved

dkrasnoff reviewed

View reviewed changes

src/main/kotlin/com/compiler/server/compiler/components/KotlinCompiler.kt Outdated Show resolved Hide resolved

dkrasnoff reviewed

View reviewed changes

src/main/kotlin/com/compiler/server/compiler/components/KotlinCompiler.kt Outdated Show resolved Hide resolved

dkrasnoff reviewed

View reviewed changes

src/main/kotlin/com/compiler/server/compiler/components/KotlinCompiler.kt Outdated Show resolved Hide resolved

dkrasnoff reviewed

View reviewed changes

src/main/kotlin/com/compiler/server/compiler/KotlinFile.kt Outdated Show resolved Hide resolved

dkrasnoff reviewed

View reviewed changes

src/main/kotlin/com/compiler/server/compiler/KotlinResolutionFacade.kt Outdated Show resolved Hide resolved

dkrasnoff reviewed

View reviewed changes

src/main/kotlin/com/compiler/server/service/KotlinProjectExecutor.kt Outdated Show resolved Hide resolved

dkrasnoff reviewed

View reviewed changes

src/main/kotlin/com/compiler/server/service/KotlinProjectExecutor.kt Outdated Show resolved Hide resolved

Collaborator

dkrasnoff commented Oct 6, 2025

@zwiora, before fixing my comments, please:

Squash all the changes starting from commit with hashbb94bb34 into one commit.
Recreate this branch based on fresh dkrasnov/KTL-724-provide-the-opportunity-to-pass-kotlin-compiler-arguments
Cherry-pick this new commit from (1) to the branch created in (2) and solve conflicts.
Then solve the comments and force push the new branch to this remote branch

zwiora force-pushed the Q3/compilation-with-BTA branch 2 times, most recently from a2f38ae to ad71a95 Compare

October 9, 2025 09:32

dkrasnoff reviewed

View reviewed changes

dependencies/build.gradle.kts Outdated Show resolved Hide resolved

gradle/libs.versions.toml Outdated Show resolved Hide resolved

gradle/libs.versions.toml Outdated Show resolved Hide resolved

src/main/kotlin/com/compiler/server/compiler/components/CompilationLogger.kt Outdated Show resolved Hide resolved

src/main/kotlin/com/compiler/server/compiler/components/CompilationLogger.kt Show resolved Hide resolved

src/main/kotlin/com/compiler/server/compiler/components/KotlinCompiler.kt Outdated Show resolved Hide resolved

src/main/kotlin/com/compiler/server/service/KotlinProjectExecutor.kt Outdated Show resolved Hide resolved

src/main/kotlin/com/compiler/server/compiler/components/KotlinCompiler.kt Outdated Show resolved Hide resolved

zwiora force-pushed the Q3/compilation-with-BTA branch from ad71a95 to 1f40004 Compare

October 14, 2025 18:14

dkrasnoff reviewed

View reviewed changes

src/main/kotlin/com/compiler/server/compiler/components/KotlinCompiler.kt Show resolved Hide resolved

src/main/kotlin/com/compiler/server/compiler/components/CompilationLogger.kt Outdated Show resolved Hide resolved

zwiora force-pushed the Q3/compilation-with-BTA branch from 1f40004 to 9a68c05 Compare

October 16, 2025 15:48

KTL-2962 Compilation with BTA for JVM

c40bba4

zwiora force-pushed the Q3/compilation-with-BTA branch from 9a68c05 to c40bba4 Compare

October 17, 2025 17:00

dkrasnoff approved these changes

View reviewed changes

zwiora merged commit ee81d65 into kotlin-community/dev

2 checks passed

S-furi pushed a commit to S-furi/kotlin-compiler-server that referenced this pull request

KTL-2962 Compilation with BTA for JVM (JetBrains#876)

6946b79

# Conflicts: #	src/main/kotlin/com/compiler/server/service/KotlinProjectExecutor.kt

dkrasnoff pushed a commit that referenced this pull request

KTL-2962 Compilation with BTA for JVM (#876)

f177776

dkrasnoff pushed a commit that referenced this pull request

KTL-2962 Compilation with BTA for JVM (#876)

e58ae57

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment