-
Notifications
You must be signed in to change notification settings - Fork 393
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.
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
Get rif of IDE plugin dependencies #3073
Conversation
Everything in this commit has been copy-pasted from IDE plugin and marked as internal
|
||
@Suppress("UNCHECKED_CAST") | ||
val resolution = CompilerExtensionPointProvider.CompilerExtensionPoint( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In K1 IdePlatformKindResolution
is used to configure global caches to analyze code for different platforms. It contains platform factories, they are used in IdeaResolverForProject
and in the inheritors of ResolverForModuleFactory
for different platforms to create some components for the frontend: biltins, package providers for Klib'as and fabrics for creating classes...
But IdeaResolverForProject
is not used in Dokka,
KotlinCacheService
we have our own stub,
ResolverForModuleFactory
we have our own inheritors, where PlatformKindResolution
is not called.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nicely done 👍
...lin/org/jetbrains/dokka/analysis/kotlin/descriptors/ide/IdeCompilerExtensionPointProvider.kt
Outdated
Show resolved
Hide resolved
...e/src/main/kotlin/org/jetbrains/dokka/analysis/kotlin/descriptors/ide/IdeDescriptorFinder.kt
Outdated
Show resolved
Hide resolved
893a4b6
to
13a7629
Compare
* Get rif of IDE plugin dependencies Everything in this commit has been copy-pasted from IDE plugin and marked as internal (cherry picked from commit 298065f)
Everything in this commit has been copy-pasted from IDE plugin and marked as internal