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

Support 2024.1 IDEs #3567

Closed
wants to merge 3 commits into from
Closed

Support 2024.1 IDEs #3567

wants to merge 3 commits into from

Conversation

junioroo
Copy link

@junioroo junioroo commented Apr 4, 2024

No description provided.

@junioroo junioroo mentioned this pull request Apr 5, 2024
@ashleysommer
Copy link
Contributor

ashleysommer commented Apr 7, 2024

@junioroo
Did you run the test suite with these changes?
I've been working on implementing 2024.1 support for this plugin myself, to learn more about writing IntelliJ plugins (I didn't see that you'd already opened this PR).

When I try to build this plugin against 2024.1 IDEs and run the Test suite, I get the error NoSuchFieldError: FILE_HASHING_STRATEGY. See related IntelliJ tracker bug for it.

While in the process of fixing that issue, I upgraded the org.jetbrains.intellij gradle plugin to v1.17.3 as recommended in the IntelliJ docs. Unfortunately that brings in some other gradle dependencies that require gradle v7.6.4 see here for details.

I notice also you have not added "2024.1" to the PluginVerifier version matrix.

So I'm currently in the process of writing a new PR that bumps support for IDE 2024.1, correctly adds the version in the PluginVerifier, fixes the NoSuchFieldError, bumps org.jetbrains.intellij gradle plugin to v1.17.3, and bumps gradlewrapper version to v7.6.4.

@ashleysommer
Copy link
Contributor

@AlexsandroBezerra @junioroo
I keep running into more issues while testing the new build on v2024.1 IDEs.
My fixes are being cumulatively added to this PR: #3569

@junioroo junioroo closed this Apr 9, 2024
@junioroo
Copy link
Author

junioroo commented Apr 9, 2024

closed in favor of:
#3569

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.

None yet

3 participants