-
Notifications
You must be signed in to change notification settings - Fork 890
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
V11 - Remove functions node bundle (#8507)
With the removal of fetch we no long need to create a node bundle for functions. Instead the sourcebase may become isomorphic, so long as we remove the older node sources. And that's what the PR does!
- Loading branch information
1 parent
7194d9c
commit d253484
Showing
7 changed files
with
16 additions
and
96 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
--- | ||
'@firebase/functions-compat': patch | ||
'@firebase/functions': patch | ||
--- | ||
|
||
Remove node bundle from the functions SDK as the node-specific fetch code has been removed in favor of using native fetch throughout the SDK. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,16 +3,16 @@ | |
"version": "0.3.14", | ||
"description": "", | ||
"author": "Firebase <[email protected]> (https://firebase.google.com/)", | ||
"main": "dist/index.node.cjs.js", | ||
"main": "dist/index.cjs.js", | ||
"browser": "dist/index.esm2017.js", | ||
"module": "dist/index.esm2017.js", | ||
"esm5": "dist/index.esm5.js", | ||
"exports": { | ||
".": { | ||
"types": "./dist/src/index.d.ts", | ||
"node": { | ||
"import": "./dist/node-esm/index.node.esm.js", | ||
"require": "./dist/index.node.cjs.js" | ||
"require": "./dist/index.cjs.js", | ||
"import": "./dist/index.esm2017.js" | ||
}, | ||
"esm5": "./dist/index.esm5.js", | ||
"browser": { | ||
|
@@ -57,7 +57,7 @@ | |
"test:all": "run-p --npm-path npm test:browser test:node", | ||
"test:browser": "karma start", | ||
"test:browser:debug": "karma start --browsers=Chrome --auto-watch", | ||
"test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha 'src/{,!(browser)/**/}*.test.ts' --file src/index.node.ts --config ../../config/mocharc.node.js", | ||
"test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha 'src/{,!(browser)/**/}*.test.ts' --config ../../config/mocharc.node.js", | ||
"test:emulator": "env FIREBASE_FUNCTIONS_HOST=http://localhost FIREBASE_FUNCTIONS_PORT=5005 run-p --npm-path npm test:node", | ||
"trusted-type-check": "tsec -p tsconfig.json --noEmit", | ||
"add-compat-overloads": "ts-node-script ../../scripts/build/create-overloads.ts -i ../functions/dist/functions-public.d.ts -o dist/src/index.d.ts -a -r Functions:types.FirebaseFunctions -r FirebaseApp:FirebaseAppCompat --moduleToEnhance @firebase/functions" | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,16 +3,16 @@ | |
"version": "0.11.8", | ||
"description": "", | ||
"author": "Firebase <[email protected]> (https://firebase.google.com/)", | ||
"main": "dist/index.node.cjs.js", | ||
"main": "dist/index.cjs.js", | ||
"browser": "dist/index.esm2017.js", | ||
"module": "dist/index.esm2017.js", | ||
"esm5": "dist/index.esm.js", | ||
"exports": { | ||
".": { | ||
"types": "./dist/functions-public.d.ts", | ||
"node": { | ||
"import": "./dist/esm-node/index.node.esm.js", | ||
"require": "./dist/index.node.cjs.js" | ||
"import": "./dist/index.esm2017.js", | ||
"require": "./dist/index.cjs.js" | ||
}, | ||
"esm5": "./dist/index.esm.js", | ||
"browser": { | ||
|
@@ -38,7 +38,7 @@ | |
"test:all": "run-p --npm-path npm test:browser test:node", | ||
"test:browser": "karma start", | ||
"test:browser:debug": "karma start --browsers=Chrome --auto-watch", | ||
"test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha 'src/{,!(browser)/**/}*.test.ts' --file src/index.node.ts --config ../../config/mocharc.node.js", | ||
"test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha 'src/{,!(browser)/**/}*.test.ts' --file src/index.ts --config ../../config/mocharc.node.js", | ||
"test:emulator": "env FIREBASE_FUNCTIONS_EMULATOR_ORIGIN=http://localhost:5005 run-p --npm-path npm test:node", | ||
"trusted-type-check": "tsec -p tsconfig.json --noEmit", | ||
"api-report": "api-extractor run --local --verbose", | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.