Skip to content
This repository has been archived by the owner on Oct 18, 2022. It is now read-only.

Commit

Permalink
use npm run dist in workflow
Browse files Browse the repository at this point in the history
  • Loading branch information
emlys committed Feb 18, 2021
1 parent f107c1d commit b362282
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 18 deletions.
18 changes: 2 additions & 16 deletions .github/workflows/build-electron.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,28 +31,14 @@ jobs:
- name: Fetch InVEST Binaries
run: npm run fetch-invest

- name: NPM Install
run: npm install

- name: Run the build script
run: npm run build

- name: Run electron-builder on macOS
if: matrix.os == 'macos-latest'
env:
GH_TOKEN: env.GITHUB_TOKEN
DEBUG: electron-builder
run: |
# use the built version as the entrypoint
# make sure we don't publish to github releases
npx --no-install electron-builder build --mac -c.extraMetadata.main=build/main.js --publish never
- name: Run electron-builder on Windows
if: matrix.os == 'windows-latest'
- name: Run electron-builder
env:
GH_TOKEN: env.GITHUB_TOKEN
DEBUG: electron-builder
run: npx --no-install electron-builder build --windows -c.extraMetadata.main=build/main.js --publish never
run: npm run dist

# Also run all tests on the build dir code?
- name: Test flask app binaries
Expand Down
3 changes: 1 addition & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,7 @@
"postinstall": "electron-builder install-app-deps",
"fetch-invest": "node ./scripts/fetch_invest_binaries.js && unzip ./build/binaries.zip -d build/invest/",
"build": "node ./build.js",
"dist": "electron-builder -c.extraMetadata.main=build/main.js",
"pack": "electron-builder --dir -c.extraMetadata.main=build/main.js"
"dist": "electron-builder build -c.extraMetadata.main=build/main.js --publish never"
},
"build": {
"extraResources": [
Expand Down

0 comments on commit b362282

Please sign in to comment.