leinardi / mypy-pycharm

A plugin providing both real-time and on-demand scanning of Python files with Mypy from within PyCharm/IDEA.
Apache License 2.0
188 stars 30 forks source link

Update deprecated notifications API usage #93

Closed intgr closed 2 years ago

intgr commented 2 years ago

Contributor checklist


Description

Docs: https://plugins.jetbrains.com/docs/intellij/notifications.html#top-level-notifications-balloons

Type of Changes

Type
:hammer: Refactoring
leinardi commented 2 years ago

Hi @intgr, looks like this changes break compatibility with older version of IDEA: image

To keep them we should raise the min version from 201.8743 to 212.5712.

What do you think?

intgr commented 2 years ago

Ah this isn't something I considered, maybe I should have.

Personally, I would just bump the minimum version and not worry about compatibility. If users don't want to upgrade to newer IDE versions, maybe they shouldn't expect to be able to update plugins either -- they can still use the old plugin version. Similar to how we did in https://github.com/leinardi/mypy-pycharm/pull/77#issuecomment-987157715

Although 212.5712 corresponds to IDEA 2021.2.4, which is still quite new.

Perhaps add a message to the plugin's description or release notes about this: if there are any users who need this plugin in older IDE versions, they can open an issue and we can consider reversing this policy.

intgr commented 2 years ago

By the way, you can change the minimum version in the JetBrains Marketplace website directly.

There is no need to make a new release to change it.

leinardi commented 2 years ago

Interesting! I'll try :)