Polish auto create layout warning color

vercel · Oct 21, 2023 · f8d3099 · f8d3099
1 parent e6fbeab
commit f8d3099
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 14 deletions.
diff --git a/packages/next/src/build/webpack/loaders/next-app-loader.ts b/packages/next/src/build/webpack/loaders/next-app-loader.ts
@@ -6,7 +6,7 @@ import path from 'path'
 import { stringify } from 'querystring'
 import { bold } from '../../../lib/picocolors'
 import { getModuleBuildInfo } from './get-module-build-info'
-import { verifyRootLayout } from '../../../lib/verifyRootLayout'
+import { verifyRootLayout } from '../../../lib/verify-root-layout'
 import * as Log from '../../output/log'
 import { APP_DIR_ALIAS, WEBPACK_RESOURCE_QUERIES } from '../../../lib/constants'
 import {

diff --git a/packages/next/src/lib/verifyRootLayout.ts → packages/next/src/lib/verify-root-layout.ts b/packages/next/src/lib/verifyRootLayout.ts → packages/next/src/lib/verify-root-layout.ts
@@ -1,6 +1,7 @@
 import path from 'path'
+import * as Log from '../build/output/log'
 import { promises as fs } from 'fs'
-import { bold, green } from './picocolors'
+import { bold } from './picocolors'
 import { APP_DIR_ALIAS } from './constants'
 
 const globOrig =
@@ -123,14 +124,12 @@ export async function verifyRootLayout({
       )
       await fs.writeFile(rootLayoutPath, getRootLayout(hasTsConfig))
 
-      console.log(
-        green(
-          `\nYour page ${bold(
-            `app/${normalizedPagePath}`
-          )} did not have a root layout. We created ${bold(
-            `app${rootLayoutPath.replace(appDir, '')}`
-          )} for you.`
-        ) + '\n'
+      Log.warn(
+        `Your page ${bold(
+          `app/${normalizedPagePath}`
+        )} did not have a root layout. We created ${bold(
+          `app${rootLayoutPath.replace(appDir, '')}`
+        )} for you.`
       )
 
       // Created root layout

diff --git a/test/e2e/app-dir/create-root-layout/create-root-layout.test.ts b/test/e2e/app-dir/create-root-layout/create-root-layout.test.ts
@@ -25,10 +25,6 @@ describe('app-dir create root layout', () => {
                 path.join(__dirname, 'next.config.js')
               ),
             },
-            dependencies: {
-              react: 'latest',
-              'react-dom': 'latest',
-            },
           })
         })
         afterAll(() => next.destroy())