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 IntelliJ IDE 2024.2 #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

honhimW
Copy link

@honhimW honhimW commented Sep 27, 2024

  • Upgrade gradle to 8.10.2;
  • Change gradle build script to Kotlin DSL;
  • Updated the Gradle Jetbrains Plugin from org.jetbrains.intellij to org.jetbrains.intellij.platform.
  • Bump JDK 21;

Note: Publishing & Signing are not tested

@honhimW honhimW changed the title build: upgrade gradle 8.10.2, ide 2024.2 Support IntelliJ IDE 2024.2 Sep 27, 2024
@aragalie
Copy link

aragalie commented Oct 1, 2024

@honhimW it seems that this plugin is unfortunately not supported anymore.

As I see that you've forked it yourself, would you by any chance also be willing to submit it to the JetBrains marketplace as a new plugin, for others can use it as well? 🙏

@honhimW
Copy link
Author

honhimW commented Oct 1, 2024

@aragalie Thanks for your suggestion.

it seems that this plugin is unfortunately not supported anymore.

Sadly indeed...

would you by any chance also be willing to submit it to the JetBrains marketplace as a new plugin.

Since I don’t have experience publishing JetBrains plugins, I’m not sure if I can publish the plugin myself (though it shouldn’t be too difficult).

However, the main issue is that I don’t fully understand how this plugin works, so maintaining a fork would be challenging for me. Additionally, this repository is under the ron-rs organization rather than the original author’s account, so it would be better to contact the organization maintainer(@juntyr) to coordinate the current status of this repository.

Lastly, I noticed that the plugin doesn’t work perfectly in practice. For example, it shows an error when enum type names contain numbers.

@honhimW
Copy link
Author

honhimW commented Oct 1, 2024

@aragalie
Additionally, I have just created a Release. If you are interested, you can manually install and try it out.

@juntyr
Copy link
Member

juntyr commented Oct 1, 2024

Thanks for the ping but I'm afraid I can't help much myself. I can merge PRs in this repo, but I've never worked on the IntelliJ plugin myself and have no idea how the publishing would work.

Perhaps @JonahPlusPlus could help with that?

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.

3 participants