ci: add step to deploy firebase functions on merge

This change is intended to automatically deploy firebase functions in
addition to the already deployed hosted content on merge.

Author: sdowell

.github/workflows/checkSite.yml

@@ -17,9 +17,11 @@ on:
   pull_request:
     paths:
       - "site/**"
+      - "firebase/**"
   push:
     paths:
       - "site/**"
+      - "firebase/**"
 jobs:
   build:
     runs-on: ubuntu-latest
@@ -36,6 +38,8 @@ jobs:
       - name: Lint site content
         run: npm i && npm run lint-check
         working-directory: site
+      - name: Lint firebase functions
+        run: make site-functions-lint
       - name: Install Site Checker
         run: yarn install
         working-directory: href-checker

.github/workflows/firebase-hosting-merge.yml

@@ -28,10 +28,16 @@ jobs:
       - uses: actions/setup-node@v2
         with:
           node-version: '16'
+      - name : Install firebase-tools
+        run: npm install -g firebase-tools
       - name: Generate Site
         run: make site-run-server
       - name: Generate sitemap
         run: make site-map
+      - name: Deploy firebase functions
+        run: make site-deploy-functions
+        env:
+          FIREBASE_TOKEN: "${{ secrets.FIREBASE_SERVICE_ACCOUNT_KPT_DEV }}"
       - uses: FirebaseExtended/action-hosting-deploy@276388dd6c2cde23455b30293105cc866c22282d # v0
         with:
           repoToken: "${{ secrets.GITHUB_TOKEN }}"

Makefile

@@ -116,3 +116,9 @@ site-check:
 
 site-verify-examples:
 	./scripts/verifyExamples.sh
+
+site-functions-deploy:
+	npm run --prefix=firebase/functions deploy
+
+site-functions-lint:
+	npm run --prefix=firebase/functions lint