Merge multiple log statements (#35310)

* Merge multiple log statements

It is inefficient to use multiple console.log satements, and if something is logged to the console in the middle of execution, it will be in the center of the text, making it hard to read.
This pull request merges multiple console.logs into one.

In addition, it reduces the bundle size.

* ensure formatting matches

* update test

Co-authored-by: JJ Kasper <[email protected]>

Author: Exortions

packages/create-next-app/index.ts

@@ -88,17 +88,14 @@ async function run(): Promise<void> {
   }
 
   if (!projectPath) {
-    console.log()
-    console.log('Please specify the project directory:')
-    console.log(
-      `  ${chalk.cyan(program.name())} ${chalk.green('<project-directory>')}`
-    )
-    console.log()
-    console.log('For example:')
-    console.log(`  ${chalk.cyan(program.name())} ${chalk.green('my-next-app')}`)
-    console.log()
     console.log(
-      `Run ${chalk.cyan(`${program.name()} --help`)} to see all options.`
+      '\nPlease specify the project directory:\n' +
+        `  ${chalk.cyan(program.name())} ${chalk.green(
+          '<project-directory>'
+        )}\n` +
+        'For example:\n' +
+        `  ${chalk.cyan(program.name())} ${chalk.green('my-next-app')}\n\n` +
+        `Run ${chalk.cyan(`${program.name()} --help`)} to see all options.`
     )
     process.exit(1)
   }
@@ -172,20 +169,17 @@ async function notifyUpdate(): Promise<void> {
     const res = await update
     if (res?.latest) {
       const pkgManager = getPkgManager()
-
-      console.log()
-      console.log(
-        chalk.yellow.bold('A new version of `create-next-app` is available!')
-      )
       console.log(
-        'You can update by running: ' +
+        chalk.yellow.bold('A new version of `create-next-app` is available!') +
+          '\n' +
+          'You can update by running: ' +
           chalk.cyan(
             pkgManager === 'yarn'
               ? 'yarn global add create-next-app'
               : `${pkgManager} install --global create-next-app`
-          )
+          ) +
+          '\n'
       )
-      console.log()
     }
     process.exit()
   } catch {
@@ -201,8 +195,10 @@ run()
     if (reason.command) {
       console.log(`  ${chalk.cyan(reason.command)} has failed.`)
     } else {
-      console.log(chalk.red('Unexpected error. Please report it as a bug:'))
-      console.log(reason)
+      console.log(
+        chalk.red('Unexpected error. Please report it as a bug:') + '\n',
+        reason
+      )
     }
     console.log()
 

test/integration/client-navigation/test/index.test.js

@@ -1716,22 +1716,21 @@ describe('Client Navigation', () => {
 
   it('should navigate to paths relative to the current page', async () => {
     const browser = await webdriver(context.appPort, '/nav/relative')
-    await browser.waitForElementByCss('body', 500)
     let page
 
     await browser.elementByCss('a').click()
 
-    browser.waitForElementByCss('#relative-1', 500)
+    browser.waitForElementByCss('#relative-1')
     page = await browser.elementByCss('body').text()
     expect(page).toMatch(/On relative 1/)
     await browser.elementByCss('a').click()
 
-    browser.waitForElementByCss('#relative-2', 500)
+    browser.waitForElementByCss('#relative-2')
     page = await browser.elementByCss('body').text()
     expect(page).toMatch(/On relative 2/)
 
     await browser.elementByCss('button').click()
-    browser.waitForElementByCss('#relative', 500)
+    browser.waitForElementByCss('#relative')
     page = await browser.elementByCss('body').text()
     expect(page).toMatch(/On relative index/)