Merge pull request #1506 from Expensify/andrew-desktop-script

Move deploy specifics to github actions for desktop builds
Expensify · Feb 19, 2021 · e9eefcf · e9eefcf
2 parents b57e27c + 5d35455
commit e9eefcf
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 825 deletions.
diff --git a/.github/workflows/desktop.yml b/.github/workflows/desktop.yml
@@ -29,7 +29,7 @@ jobs:
                   DEVELOPER_ID_SECRET_PASSPHRASE: ${{ secrets.DEVELOPER_ID_SECRET_PASSPHRASE }}
 
             - name: Build desktop app
-              run: npm run desktop-build
+              run: npm run desktop-build -- --publish always -c.afterSign=desktop/notarize.js
               env:
                   CSC_LINK: ${{ secrets.CSC_LINK }}
                   CSC_KEY_PASSWORD: ${{ secrets.CSC_KEY_PASSWORD }}

diff --git a/config/electron.config.js b/config/electron.config.js
@@ -4,7 +4,6 @@ module.exports = {
     extraMetadata: {
         main: './desktop/main.js',
     },
-    afterSign: 'desktop/notarize.js',
     mac: {
         category: 'public.app-category.finance',
         icon: './desktop/icon.png',