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

Show update notification #498

Closed FWDekker closed 10 months ago

FWDekker commented 10 months ago

I had hoped to implement #481, but on reflection that is too difficult; it would require adding back in code to parse symbol sets, read external files as dictionaries, and more, which would take dozens of lines which would need debugging. Instead, I think it's safer to ignore the older settings and show a notification informing the user that this has happened.

The notification will only be shown once. There are currently no plans to show notifications for each update; I think doing so would hinder the user. However, this specific upgrade is major, and I think it therefore warrants a separate notification.

I had considered detecting whether the user is upgrading from v2, but doing so would require quite a bit of code which I'm not sure is stable, so I have decided to forego that. That has now been implemented.

codecov[bot] commented 10 months ago

Codecov Report

Merging #498 (1342557) into main (691b654) will increase coverage by 0.05%. Report is 2 commits behind head on main. The diff coverage is 68.18%.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/FWDekker/intellij-randomness/pull/498/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/498?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 #498 +/- ## ========================================== + Coverage 85.87% 85.92% +0.05% ========================================== Files 51 52 +1 Lines 1940 1990 +50 Branches 333 334 +1 ========================================== + Hits 1666 1710 +44 - Misses 213 217 +4 - Partials 61 63 +2 ```