Skip to content
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

build: Upgrade to latest Electron stable #106

Merged
merged 10 commits into from Jul 17, 2023
Merged

build: Upgrade to latest Electron stable #106

merged 10 commits into from Jul 17, 2023

Conversation

JohanDelao
Copy link
Contributor

@JohanDelao JohanDelao commented Jul 12, 2023

Description

Sleuth is currently running on Electron v13.3.0, which is not a stable version (see here). To stay up to date with security best practices, bug fixes, performance enhancements and future maintenance, upgrading Sleuth to the latest stable version is needed.

This PR upgrades Sleuth to Electron v25.2.0 (latest at this time) and fixes #68

Notes

  • When loading up Sleuth in DevMode, a white window will appear for a few seconds before Sleuth appears. This doesn't appear in production and doesn't impact performance.
  • Removed app.allowRendererProcessReuse as it is no longer supported and was removed in Electron v14.0.0
  • Removed cache from type name as it was removed from app.getPath here
  • Removed new-window function due to new-window event being deprecated in Electron 13 and removed in Electron 22
  • Removed ARE_YOU_BUSY IPC Event and associated functions due to being unused and dead-code

@JohanDelao JohanDelao marked this pull request as ready for review July 14, 2023 21:19
@erickzhao
Copy link
Member

erickzhao commented Jul 17, 2023

Merged in yarn.lock conflict resolution created by #102

@erickzhao erickzhao merged commit f2cf702 into main Jul 17, 2023
5 checks passed
@erickzhao erickzhao deleted the electronUpgrade branch July 17, 2023 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade to latest Electron stable
2 participants