Skip to content

[BTA] KT-86781 Deprecate FIR based incremental runner#6184

Open
Wojtek Kaliciński (wojtek-kalicinski) wants to merge 1 commit into
masterfrom
wkal/bta_deprecate_fir
Open

[BTA] KT-86781 Deprecate FIR based incremental runner#6184
Wojtek Kaliciński (wojtek-kalicinski) wants to merge 1 commit into
masterfrom
wkal/bta_deprecate_fir

Conversation

@wojtek-kalicinski

Copy link
Copy Markdown
Contributor

No description provided.

@kotlin-safemerge

kotlin-safemerge Bot commented Jun 9, 2026

Copy link
Copy Markdown

Code Owners

RuleOwnersApproval
/​compiler/​build-​tools/​
build-tools-api
🔴
@ALikhachev

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd suggest also printing a deprecation warning from the IncrementalFirJvmCompilerRunner itself, so not only the configuration produces it but the actual usage too (so that not only integration authors notice it, but their users too)

*/
@JvmField
@ExperimentalCompilerArgument
@Deprecated("The FIR runner is deprecated and will be removed in a future version.")

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we specify the exact planned version? Conservatively, if we deprecate it in 2.4.20, it should become an error in 2.6.0 and removed in 2.7.0
More aggressively (since it's explicitly experimental), probably we could make it an error in 2.5.0 and remove in 2.6.0

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants