[FIR] Allow companion property with initializer in interfaces KT-85770#5881
[FIR] Allow companion property with initializer in interfaces KT-85770#5881KotlinBuild merged 1 commit intomasterfrom
Conversation
Code Owners
|
| @@ -0,0 +1,11 @@ | |||
| // RUN_PIPELINE_TILL: BACKEND | |||
There was a problem hiding this comment.
If there are no errors, I'd prefer to have this as a codegen box test instead, and also check that access to such property (both get and set) is generated correctly.
There was a problem hiding this comment.
Hmmm, so why are we merging support for something that doesn't work instead of making it work first?
There was a problem hiding this comment.
Reasonable question. IMO, experimental features always need to exist in an incomplete state in master unless we want to drop them in one giant PR spanning multiple subsystems.
Case in point; the feature basically only works on the JVM right now.
Here, we're fixing the frontend part of this bug. The backend part has a separate ticket and will probably be fixed by your team. Unless we start doing stacked PRs, I don't see a nicer way to deliver these fixes.
|
/safe-merge |
|
Quality gate is triggered at https://buildserver.labs.intellij.net/build/930243934 — use this link to get full insight. Quality gate was triggered with the following revisions:
|
|
Quality gate finished successfully. |
4 participants
No description provided.