From d4bef88b2b8132b5b892ddf31b5944871dd5f31e Mon Sep 17 00:00:00 2001 From: Sophia Willows <20146550+sophiabits@users.noreply.github.com> Date: Fri, 18 Feb 2022 11:22:52 +1300 Subject: [PATCH] Only log experiments warning if user actually opted in to an experiment(s) (#34413) Currently if you have a Next config like the following: ```js module.exports = { experimental: {}, }, ``` You are presented with the warning for experimental features, even though you haven't actually enabled any experiments. This PR checks there's at least one key in the `experimental` object before logging the warning. --- packages/next/server/config.ts | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/packages/next/server/config.ts b/packages/next/server/config.ts index 1250ea0dfe38004..2e334084a915830 100644 --- a/packages/next/server/config.ts +++ b/packages/next/server/config.ts @@ -67,8 +67,9 @@ function assignDefaults(userConfig: { [key: string]: any }) { if ( key === 'experimental' && - value !== undefined && - value !== defaultConfig[key] + value !== defaultConfig[key] && + typeof value === 'object' && + Object.keys(value).length > 0 ) { experimentalWarning() }