Skip to content

Commit 8960b47

Browse files
committed
Merging in master
1 parent 5fe4498 commit 8960b47

File tree

2 files changed

+13
-1
lines changed

2 files changed

+13
-1
lines changed

src/deploy/functions/prepare.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,11 @@ export async function prepare(
100100
const targetedCodebaseConfigs = context.config!.filter((cfg) => codebases.includes(cfg.codebase));
101101

102102
// Load runtime config if API is enabled and at least one targeted codebase uses it
103-
if (checkAPIsEnabled[1] && targetedCodebaseConfigs.some(shouldUseRuntimeConfig)) {
103+
if (
104+
allowFunctionsConfig &&
105+
checkAPIsEnabled[1] &&
106+
targetedCodebaseConfigs.some(shouldUseRuntimeConfig)
107+
) {
104108
runtimeConfig = { ...runtimeConfig, ...(await getFunctionsConfig(projectId)) };
105109
}
106110

src/experiments.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,14 @@ export const ALL_EXPERIMENTS = experiments({
5757
"of how that image was created.",
5858
public: false,
5959
},
60+
legacyRuntimeConfigCommands: {
61+
shortDescription: "Expose legacy functions.config() CLI commands",
62+
fullDescription:
63+
"The Cloud Runtime Config API is deprecated. Enable this experiment to continue using the " +
64+
"`functions:config:*` commands while you migrate to the Firebase Functions params APIs.",
65+
default: true,
66+
public: true,
67+
},
6068
runfunctions: {
6169
shortDescription:
6270
"Functions created using the V2 API target Cloud Run Functions (not production ready)",

0 commit comments

Comments
 (0)