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

Release/5.0.0 #786

Merged
merged 8 commits into from
Feb 26, 2020
Merged

Release/5.0.0 #786

merged 8 commits into from
Feb 26, 2020

Conversation

nsakaimbo
Copy link
Contributor

Description

Merge release/v5.0.0 into develop.

nsakaimbo and others added 8 commits February 10, 2020 11:41
This PR updates the application icon folder structure and declarations to be aligned with updated electron-builder documentation. Specifically, we streamline the usage of application icons by:

- Removing direct icon references for each platform in package.json, and replacing with a top-level buildResources directory
- Removing the linux-specific icon so electron-builder can use icon.icns for both Linux and Mac (per the docs)

Per the docs, icons in buildResources will automatically be detected and used accordingly as follows:

- icon.icns -- Mac and Linux (installers and applications)
- background.png and [email protected] (Mac installer backgrounds)
- icon.ico -- Windows (installer and application)
* Revert workaround for buggy after_sign_hook

* Bump version to 5.0.0-beta3
@nsakaimbo nsakaimbo requested a review from loremattei February 26, 2020 19:01
@nsakaimbo nsakaimbo merged commit 6fc6e7b into develop Feb 26, 2020
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.

2 participants