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
Patch emulator node runtime discovery to favor local cache
  • Loading branch information
@abeisgoat
abeisgoat committed Oct 23, 2020
commit 7f7f8c24867745e25b48065cf4c47c42bab40911
16 changes: 8 additions & 8 deletions src/emulator/functionsEmulator.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -707,24 +707,24 @@ export class FunctionsEmulator implements EmulatorInstance {
// Will happen if we haven't asked about local version yet
}

// If the requested version is the same as the host, let's use that
if (requestedMajorVersion === hostMajorVersion) {
// If the requested version is already locally available, let's use that
if (requestedMajorVersion === localMajorVersion) {
this.logger.logLabeled(
"SUCCESS",
"functions",
`Using node@${requestedMajorVersion} from host.`
`Using node@${requestedMajorVersion} from local cache.`
);
return process.execPath;
return localNodePath;
}

// If the requested version is already locally available, let's use that
if (localMajorVersion === requestedMajorVersion) {
// If the requested version is the same as the host, let's use that
if (requestedMajorVersion === hostMajorVersion) {
this.logger.logLabeled(
"SUCCESS",
"functions",
`Using node@${requestedMajorVersion} from local cache.`
`Using node@${requestedMajorVersion} from host.`
);
return localNodePath;
return process.execPath;
}

// Otherwise we'll begin the conversational flow to install the correct version locally
Expand Down