search

OpenLiberty / liberty-tools-intellij

IntelliJ IDEA extension for Liberty
https://plugins.jetbrains.com/plugin/14856-open-liberty-tools
Eclipse Public License 2.0
12 stars 24 forks source link

Jakarta EE quick fixes: Adding a catch and rethrow for exceptions that should be propagated up the call stack. #750

Closed mrglavas closed 4 months ago

mrglavas commented 4 months ago

Added a catch/rethrow of IndexNotReadyException, ProcessCanceledException, CancellationException to resolveCodeAction() and related methods. Resolves https://github.com/OpenLiberty/liberty-tools-intellij/issues/635.

Also cleaned up imports and updated copyrights.

mrglavas commented 4 months ago

As discussed earlier, the duplicate code is OK for now, but we should look to refactor the common code for resolveCodeAction() in the future.

Agreed and we should look at this more holistically than this one example. I opened: https://github.com/OpenLiberty/liberty-tools-intellij/issues/754 to track this activity.