Skip to content

Commit 4408aeb

Browse files
authored
Fix bug where emulator didn't consider .env.local when discovering triggers. (#5477)
Fixes #5219. Note that the actual fix is a one-liner. I'm adding basic integration test for dotenv and params support for the emulator.
1 parent 020c96b commit 4408aeb

File tree

5 files changed

+824
-594
lines changed

5 files changed

+824
-594
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,3 +5,4 @@
55
- Pass `trailingSlash` from Next.js config to `firebase.json` (#5445)
66
- Don't use Next.js internal redirects for the backend test (#5445)
77
- Fix issue where pnpm support broke for function emulation and deployment. (#5467)
8+
- Fix bug where .env.local files were not picked up during function emulation. (#5477)

scripts/emulator-tests/functions/package-lock.json

Lines changed: 9 additions & 11 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

scripts/emulator-tests/functions/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
"dependencies": {
77
"express": "^4.18.1",
88
"firebase-admin": "^11.5.0",
9-
"firebase-functions": "^3.22.0"
9+
"firebase-functions": "^4.0.0"
1010
},
1111
"engines": {
1212
"node": "16"

0 commit comments

Comments
 (0)