gitify-app / gitify

GitHub notifications on your menu bar. Available on macOS, Windows & Linux.

Home Page:https://www.gitify.io

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Remove @electron/remote

bmulholland opened this issue · comments

Usage of @electron/remote is discouraged

We can replace it with inter-process communication fairly easily, just need to do it piece by piece.

@bmulholland - would you mind confirming that this can be closed. I did a search and couldn't find any references to remote from the electron package directly. All seem to have been migrated to @electron/remote (thank you kindly)

All seem to have been migrated to @electron/remote

That's the package that is discouraged and shouldn't be used. From the linked article, about @electron/remote itself:

⚠️ Warning! This module has many subtle pitfalls. There is almost always a better way to accomplish your task than using this module. For example, ipcRenderer.invoke can serve many common use cases.

Oops! I misread it. I focused on this comment but missed the warning. Thanks for the correction 🙇

@electron/remote is a replacement for the built-in remote module in Electron, which is deprecated and will eventually be removed.