Skip to content

Added more targets for DisableInterpretation to help combine use of plugin together with library code#1790

Open
koperagen wants to merge 1 commit intomasterfrom
disable-dataframe-plugin
Open

Added more targets for DisableInterpretation to help combine use of plugin together with library code#1790
koperagen wants to merge 1 commit intomasterfrom
disable-dataframe-plugin

Conversation

@koperagen
Copy link
Copy Markdown
Collaborator

We need a way to make this code compile and work with plugin enabled in the project. Now it doesn't because declaring local classes is impossible in inline

inline fun <reified T> DataFrame<T>.doSomething(): DataFrame<T> { 
 ...
}

Let's add more targets to DisableInterpretation:

@DisableInterpretation
inline fun <reified T> DataFrame<T>.doSomething(): DataFrame<T> { 
 ...
}

@koperagen koperagen added this to the 1.0.0-Beta5 milestone Apr 3, 2026
@koperagen koperagen requested a review from Jolanrensen April 3, 2026 15:36
@koperagen koperagen self-assigned this Apr 3, 2026
@koperagen koperagen added the Compiler plugin Anything related to the DataFrame Compiler Plugin label Apr 3, 2026
@koperagen koperagen force-pushed the disable-dataframe-plugin branch from 41a2007 to b2104c4 Compare April 3, 2026 16:28
@Jolanrensen
Copy link
Copy Markdown
Collaborator

seems useful indeed :)

@Jolanrensen
Copy link
Copy Markdown
Collaborator

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

Labels

Compiler plugin Anything related to the DataFrame Compiler Plugin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants