Skip to content
This repository has been archived by the owner on May 21, 2019. It is now read-only.

Updates package script to run gulp build before electron-packager #148

Merged
merged 1 commit into from
Sep 29, 2015
Merged

Updates package script to run gulp build before electron-packager #148

merged 1 commit into from
Sep 29, 2015

Conversation

curiouslychase
Copy link
Contributor

Proposing a solution for #32.

It seems like the tasks for compiling typescript, react and sass isn't run before npm run package, which is what's causing this issue, if you haven't run npm start yet.

This is a possible scenario that works.

@luizkowalski @jbaskeen @mzgnr

@0zguner 0zguner self-assigned this Sep 24, 2015
@sergeysova
Copy link

+1

1 similar comment
@G07cha
Copy link
Contributor

G07cha commented Sep 28, 2015

+1

vlad-shatskyi added a commit that referenced this pull request Sep 29, 2015
Updates package script to run gulp build before electron-packager
@vlad-shatskyi vlad-shatskyi merged commit 70cfa49 into railsware:master Sep 29, 2015
@vlad-shatskyi
Copy link
Contributor

@chaseadamsio, thank you for the contribution.

@curiouslychase
Copy link
Contributor Author

👍

@curiouslychase curiouslychase deleted the fix/npm-run-package branch September 29, 2015 14:53
@curiouslychase
Copy link
Contributor Author

👍 happy to contribute to something that displaces the antiquated tools we have to deal with now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants