Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Swap emulator Node runtime discovery to favor local cache #2740

Merged
merged 12 commits into from
Oct 28, 2020
Merged

Swap emulator Node runtime discovery to favor local cache #2740

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
b7fb744
Fixes passing flags to firepit node shim
abeisgoat Jun 2, 2020
331233c
Remove indent
abeisgoat Jun 2, 2020
a0543fb
Adds comments
abeisgoat Jun 2, 2020
d5746f4
Merge branch 'master' of https://github.com/firebase/firebase-tools
abeisgoat Oct 23, 2020
7f7f8c2
Patch emulator node runtime discovery to favor local cache
abeisgoat Oct 23, 2020
34d34aa
Adds warning when attempting to inspect functions under firepit
abeisgoat Oct 23, 2020
76588b4
Changelog
abeisgoat Oct 28, 2020
e7a7679
Simplify code
abeisgoat Oct 28, 2020
d825c18
Merge branch 'master' into ah/functions-emulator-discover-npm-node
abeisgoat Oct 28, 2020
5266a85
Add number to changelog
abeisgoat Oct 28, 2020
3743e13
Merge branch 'ah/functions-emulator-discover-npm-node' of https://git…
abeisgoat Oct 28, 2020
29e4152
Merge branch 'master' into ah/functions-emulator-discover-npm-node
abeisgoat Oct 28, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Prev Previous commit
Next Next commit
Simplify code
  • Loading branch information
@abeisgoat
abeisgoat committed Oct 28, 2020
commit e7a7679ac67697b03b1358619be0f8e6de04a8f2
8 changes: 2 additions & 6 deletions src/emulator/functionsEmulator.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -676,13 +676,9 @@ export class FunctionsEmulator implements EmulatorInstance {
* Returns a node major version ("10", "8") or null
* @param frb the current Functions Runtime Bundle
*/
getRequestedNodeRuntimeVersion(frb: FunctionsRuntimeBundle): string | null {
getRequestedNodeRuntimeVersion(frb: FunctionsRuntimeBundle): string | undefined {
const pkg = require(path.join(frb.cwd, "package.json"));
if ((!pkg.engines || !pkg.engines.node) && !frb.nodeMajorVersion) {
return null;
}

return frb.nodeMajorVersion || pkg.engines.node;
return frb.nodeMajorVersion || (pkg.engines && pkg.engines.node);
}
/**
* Returns the path to a "node" executable to use.
Expand Down