Skip to content

Commit

Permalink
fix: use finally
Browse files Browse the repository at this point in the history
  • Loading branch information
ArnaudBarre committed Nov 10, 2022
1 parent 7ba1c3c commit d13e200
Showing 1 changed file with 9 additions and 5 deletions.
14 changes: 9 additions & 5 deletions packages/vite/src/node/cli.ts
Expand Up @@ -150,10 +150,11 @@ cli
server.printUrls()
stopProfiler((message) => server.config.logger.info(` ${message}`))
} catch (e) {
createLogger(options.logLevel).error(
colors.red(`error when starting dev server:\n${e.stack}`),
{ error: e }
)
const logger = createLogger(options.logLevel)
logger.error(colors.red(`error when starting dev server:\n${e.stack}`), {
error: e
})
stopProfiler(logger.info)
process.exit(1)
}
})
Expand Down Expand Up @@ -217,8 +218,9 @@ cli
{ error: e }
)
process.exit(1)
} finally {
stopProfiler((message) => createLogger(options.logLevel).info(message))
}
stopProfiler((message) => createLogger(options.logLevel).info(message))
})

// optimize
Expand Down Expand Up @@ -301,6 +303,8 @@ cli
{ error: e }
)
process.exit(1)
} finally {
stopProfiler((message) => createLogger(options.logLevel).info(message))
}
}
)
Expand Down

0 comments on commit d13e200

Please sign in to comment.