-
Notifications
You must be signed in to change notification settings - Fork 540
Add support for IntelliJ 2023.1 #4037
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
Conversation
I tested this locally against |
You would also need to include the new version there |
thanks, good hint! There are a bunch of compatibility errors as well, I'll take a look whether there are more things I missed. |
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.
wow, thank you for doing this. This was some very complex stuff to implement the first time
* Update testcontainers-java monorepo to v1.18.0 (#4035) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Use runTest from kotlinx.coroutines (#3674) * Add buildscans to the sample (#4036) * Add support for IntelliJ 2023.1 (#4037) * Fix accessing KotlinFindUsagesHandlerFactory on newer IDEA * Fix mutating plugin classloader on newer IDEA * Remove debug log line * Fix spotless violations * Add IDEA 2023.1 to runPluginVerifier task * Fix compatibility issues as reported by plugin verifier * Fix spotless violations * Enable local build cache (#4038) * Update kotlin monorepo to v1.8.20 * Update plugin ksp to v1.8.20-1.0.10 (#4034) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update yarn lock file * Fix the sample tests * Fix sample js tests * Fix spotless --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Veyndan Stuart <[email protected]> Co-authored-by: Philip Wedemann <[email protected]> Co-authored-by: Madis Pink <[email protected]> Co-authored-by: hfhbd <[email protected]>
Fixes #4025
Two incompatibilities:
pluginClassLoader
reflection hack didn't work any more -files
field had been removed and it looks like the functionality now depends on the internalClassPath
field. The method is quite naughty indeed :pKotlinFindUsagesHandlerFactory
changed it's package. I added a layer of indirection and some reflection on the newer versions, but there could be better strategies here.Feedback welcome as both of these are quite ugly hacks!