search

FWDekker / intellij-randomness

IntelliJ plugin for inserting random numbers, UUIDs, names, IP addresses, and much more
https://plugins.jetbrains.com/plugin/9836-randomness
MIT License
45 stars 7 forks source link

Disable dynamic plugin reloading #522

Closed FWDekker closed 8 months ago

FWDekker commented 8 months ago

Instead of fixing #520, this just disables the dynamic reloading feature.

I spent more than 6 hours trying to find out why dynamic reloading didn't work, found a bunch of supposed memory leaks, and then found extremely insufficient documentation on JetBrains' end on alternative design patterns to avoid those leaks. (Apparently every single Companion object was still referenced by the classloader. How am I supposed to avoid that?)

It's not worth my time. By explicitly disabling dynamic reloading, at least users won't see their IDE trying and failing to dynamically reload the plugin.

codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (4afe529) 85.50% compared to head (ca43a2f) 85.50%.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/FWDekker/intellij-randomness/pull/522/graphs/tree.svg?width=650&height=150&src=pr&token=HXVFGMwtwv&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Florine+W.+Dekker)](https://app.codecov.io/gh/FWDekker/intellij-randomness/pull/522?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Florine+W.+Dekker) ```diff @@ Coverage Diff @@ ## main #522 +/- ## ======================================= Coverage 85.50% 85.50% ======================================= Files 51 51 Lines 2035 2035 Branches 343 343 ======================================= Hits 1740 1740 Misses 224 224 Partials 71 71 ```