Skip to content

Commit 576b55a

Browse files
authored
Merge pull request #104 from ClayPulse/hotfix
Force electronjs build workflow to use bash shell to set env
2 parents 66d8074 + e8b6b1c commit 576b55a

File tree

2 files changed

+12
-10
lines changed

2 files changed

+12
-10
lines changed

.github/workflows/build-desktop.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ jobs:
3434
run: npm run shared-utils-build
3535

3636
- name: Save public env variables into .env file
37+
shell: bash
3738
run: |
3839
echo NEXT_PUBLIC_BACKEND_URL=$NEXT_PUBLIC_BACKEND_URL >> ./web/.env
3940
echo NEXT_PUBLIC_CDN_URL=$NEXT_PUBLIC_CDN_URL >> ./web/.env

.github/workflows/release-desktop.yml

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,17 @@ jobs:
3030
- name: Build shared-utils
3131
run: npm run shared-utils-build
3232

33+
- name: Save public env variables into .env file
34+
shell: bash
35+
run: |
36+
echo NEXT_PUBLIC_BACKEND_URL=$NEXT_PUBLIC_BACKEND_URL >> ./web/.env
37+
echo NEXT_PUBLIC_CDN_URL=$NEXT_PUBLIC_CDN_URL >> ./web/.env
38+
echo NEXT_PUBLIC_STORAGE_CONTAINER=$NEXT_PUBLIC_STORAGE_CONTAINER >> ./web/.env
39+
env:
40+
NEXT_PUBLIC_BACKEND_URL: ${{ vars.NEXT_PUBLIC_BACKEND_URL }}
41+
NEXT_PUBLIC_CDN_URL: ${{ vars.NEXT_PUBLIC_CDN_URL }}
42+
NEXT_PUBLIC_STORAGE_CONTAINER: ${{ vars.NEXT_PUBLIC_STORAGE_CONTAINER }}
43+
3344
- name: Install web dependencies
3445
run: npm install --workspace=web
3546

@@ -41,16 +52,6 @@ jobs:
4152
run: npm run rebuild-native
4253
working-directory: desktop
4354

44-
- name: Save public env variables into .env file
45-
run: |
46-
echo NEXT_PUBLIC_BACKEND_URL=$NEXT_PUBLIC_BACKEND_URL >> ./web/.env
47-
echo NEXT_PUBLIC_CDN_URL=$NEXT_PUBLIC_CDN_URL >> ./web/.env
48-
echo NEXT_PUBLIC_STORAGE_CONTAINER=$NEXT_PUBLIC_STORAGE_CONTAINER >> ./web/.env
49-
env:
50-
NEXT_PUBLIC_BACKEND_URL: ${{ vars.NEXT_PUBLIC_BACKEND_URL }}
51-
NEXT_PUBLIC_CDN_URL: ${{ vars.NEXT_PUBLIC_CDN_URL }}
52-
NEXT_PUBLIC_STORAGE_CONTAINER: ${{ vars.NEXT_PUBLIC_STORAGE_CONTAINER }}
53-
5455
- name: Build Electron App
5556
run: |
5657
npm run desktop-build

0 commit comments

Comments
 (0)