-
Notifications
You must be signed in to change notification settings - Fork 32
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
Feature request: Make the insatller too #5
Comments
cool idea @Altiano , i'll try to make same implementation next week |
great.. I'll be waiting for it 👍 |
+1 I would like this option a lot too. ! |
@vrunoa Any luck with adding in the electron-builder bit? |
@dorianpula I doubt that is it is required. electron-builder allows you to just use convention, infers a lot of data from package.json, allows to specify configuration in the package.json |
@develar Ah ok. I'll take a look at the example. I ran into some issues using plain old electron-packager which may be caused by my use of electron-prebuilt-compile (since I'd like to use some of the draft features from es2015). Using |
@dorianpula @develar sorry for the late late reply. I'm really busy with some deadlines I couldn't reach out to even read about electron-builder module. As soon as I can I would give you feedback. PR's are welcomed 😉 |
Thanks @develar I got @vrunoa having an interactive setup to configure |
if you use electron-builder, you don't need to use electron-packager.
You can always create plain old folder of icons and don't rely on icns transform.
Any possible scenario, please. Why it is more convenient than just edit your package.json by hand? |
@develar I think it's just a matter of personal preferences. Some people love interaction more than configuration in some cases. |
why not combine this with electron-builder.. so that we also have the installer..
The text was updated successfully, but these errors were encountered: