Skip to content
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

Add support for IntelliJ 2023.1 #4037

Merged
merged 8 commits into from
Apr 5, 2023

Conversation

madisp
Copy link
Contributor

@madisp madisp commented Apr 5, 2023

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 internal ClassPath field. The method is quite naughty indeed :p
  • KotlinFindUsagesHandlerFactory 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!

@madisp
Copy link
Contributor Author

madisp commented Apr 5, 2023

I tested this locally against 2.0.0-alpha05 Gradle plugin and it seems to work.

@aperfilyev
Copy link
Collaborator

You would also need to include the new version there

@madisp
Copy link
Contributor Author

madisp commented Apr 5, 2023

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.

Copy link
Collaborator

@AlecKazakova AlecKazakova left a 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

@AlecKazakova AlecKazakova merged commit 85430a3 into cashapp:master Apr 5, 2023
@madisp madisp deleted the support-ij-2023.1 branch April 5, 2023 15:48
hfhbd added a commit that referenced this pull request Apr 7, 2023
* 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 <veyndan@gmail.com>
Co-authored-by: Philip Wedemann <22521688+hfhbd@users.noreply.github.com>
Co-authored-by: Madis Pink <1147723+madisp@users.noreply.github.com>
Co-authored-by: hfhbd <hfhbd@users.noreply.github.com>
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.

Support IntelliJ 2023.1
3 participants