-
Notifications
You must be signed in to change notification settings - Fork 31
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
ActionUpdateThread.OLD_EDT
is deprecated and going to be removed soon
#120
Comments
Hello, I would like to report that I received the same error about the plugin:
Hopefully, @leinardi can dedicate some time to fixing it since the plugin is very useful. 🙏 |
Hi @fjossandon, I'll try to have a look but I really hope that more contributors will show up to help with the plugin, since I don't really have much time anymore... EDIT |
Hi @leinardi, unfortunately, I'm only familiar with Python and Perl, I have zero Java knowledge, so I cannot give more help than reporting the issue. I hope it's not too difficult to fix, many thanks for trying. |
Yep got the same thing: |
Im waiting fix too |
I fixed the
Fixing this requires a big refactoring of the logic that handles the files that should be scanned and, unfortunately, I do not have time for it. I only see 2 options for this: someone else jumps in to help with this refactoring or I just make it official that the plugin is abandoned and reached EOL. PS |
Hello again @leinardi, Then I saw in another repo, that people used Best regards, |
Hi @fjossandon, thank you for taking the time to search for possible solutions. While I was looking for a fix I ended up on the very same results:
|
How hard would a port to Kotlin be, given that there are translation tools and also ChatGPT or similar can probably do a good job? |
@kjyv are you volunteering to do it? 🙂 |
fixes leinardi#120 Signed-off-by: Dominik Willner <th33xitus@gmail.com>
I haven't really touched Java in 15 years and have no idea of Kotlin, but I might at some point the more annoyed I get by that error message. If you're adding the fix to the Kotlin version then ;) |
@kjyv Haha, I see where you're coming from! 😄 But if we're going down the Kotlin road, I'd only consider accepting a conversion if the person taking on the job would also take over the maintenance of the project. That means not just converting it, but also handling fixes like the one that's been causing annoyance. Switching languages is a pretty significant step, and while tools and AI can help, the real challenge comes in maintaining and evolving the project afterward. I'd need someone fully committed to that for the long haul. If you're up for it, though, we can definitely talk! 😉 |
I don't really want to engage in the "port to Kotlin" discussion here, as it makes no sense at this point imho. Apart from that you don't need to port a full project at once. Kotlin and Java are interoperable. So you could port pieces of the software one by one. Anyways... i provided a patch for the issue with #122. I have the plugin currently working in IDEA 2024.2.3 and at least those prominent deprecation warnings are gone. |
fixes leinardi#120 Signed-off-by: Dominik Willner <th33xitus@gmail.com>
Fix #120: Defined action update thread for actions
Thanks @dw-0 and @leinardi for the release! 🎉 |
@fjossandon I guess in a couple of business days: |
I see, thanks for the answer. I will be waiting and thanks again. |
Step 1: Are you in the right place?
Step 2: Describe your environment
Step 3: Describe the problem:
Steps to reproduce:
Observed Results:
Expected Results:
The text was updated successfully, but these errors were encountered: