Skip to content

Commit 47b99de

Browse files
authored
refactor: simplify environment variable handling for Firebase deployment (#462)
* refactor: simplify environment variable handling for Firebase deployment * refactor: update Firebase deployment jobs for pull request handling * refactor: update deployment step name and .gitignore for production functions
1 parent cfc769c commit 47b99de

File tree

2 files changed

+31
-4
lines changed

2 files changed

+31
-4
lines changed

.github/workflows/github-apps-cd.yml

Lines changed: 29 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,33 @@ jobs:
1313
firebase-functions-tests:
1414
uses: ./.github/workflows/reusable-firebase-functions-ci.yml
1515

16-
deploy-firebase-functions:
16+
deploy-firebase-functions-dev:
17+
if: github.event_name == 'pull_request'
18+
needs: [github-apps-tests, firebase-functions-tests]
19+
runs-on: ubuntu-latest
20+
defaults:
21+
run:
22+
working-directory: openci-runner/firebase
23+
steps:
24+
- uses: actions/checkout@v4
25+
26+
- uses: actions/setup-node@v4
27+
with:
28+
node-version-file: .nvmrc
29+
30+
- uses: google-github-actions/auth@v2
31+
with:
32+
credentials_json: ${{ secrets.FIREBASE_SA_KEY_FOR_PREVIEW_FUNCTIONS }}
33+
34+
- name: Install dependencies
35+
working-directory: openci-runner/firebase/functions
36+
run: npm ci
37+
38+
- name: Deploy Functions (dev)
39+
run: npx firebase-tools deploy --only functions --project ${{ secrets.FIREBASE_PROJECT_ID_FOR_PREVIEW_FUNCTIONS }}
40+
41+
deploy-firebase-functions-production:
42+
if: github.event_name == 'push'
1743
needs: [github-apps-tests, firebase-functions-tests]
1844
runs-on: ubuntu-latest
1945
defaults:
@@ -34,5 +60,5 @@ jobs:
3460
working-directory: openci-runner/firebase/functions
3561
run: npm ci
3662

37-
- name: Deploy Functions
38-
run: npx firebase-tools deploy --only functions --project ${{ secrets.FIREBASE_PROJECT_ID }}
63+
- name: Deploy Functions (Production)
64+
run: npx firebase-tools deploy --only functions --project ${{ secrets.FIREBASE_PROJECT }}

openci-runner/firebase/functions/.gitignore

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,4 +8,5 @@ typings/
88
# Node.js dependency directory
99
node_modules/
1010
*.local
11-
service_account.json
11+
service_account.json
12+
service_account_dev.json

0 commit comments

Comments
 (0)