-
Notifications
You must be signed in to change notification settings - Fork 258
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove @electron/remote #650
Comments
@bmulholland - would you mind confirming that this can be closed. I did a search and couldn't find any references to |
That's the package that is discouraged and shouldn't be used. From the linked article, about
|
Oops! I misread it. I focused on this comment but missed the warning. Thanks for the correction 🙇
|
* refactor: main js * refactor: main js * Merge remote-tracking branch 'origin/main' into refactor/mainjs * defer removal until #650 is resolved * add comments to group sections
Usage of @electron/remote is discouraged
We can replace it with inter-process communication fairly easily, just need to do it piece by piece.
The text was updated successfully, but these errors were encountered: