New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: fix minor issues in build scripts #7458
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #7458 +/- ##
=======================================
Coverage 70.32% 70.32%
=======================================
Files 88 88
Lines 3704 3704
Branches 1010 1010
=======================================
Hits 2605 2605
Misses 892 892
Partials 207 207
Continue to review full report at Codecov.
|
// Encountered an unrecoverable error | ||
case 'FATAL': | ||
this.formatError(event.error) | ||
return this.logger.fatal(event.error) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no longer a FATAL event.
@@ -27,7 +29,7 @@ async function main () { | |||
// Step 1: Apply suffixes | |||
for (const pkg of workspacePackages) { | |||
if (pkg.options.suffix && pkg.options.suffix.length) { | |||
await pkg.suffixAndVersion() | |||
pkg.suffixAndVersion() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not an async method.
exec (command, args, silent = false) { | ||
const r = spawn.sync(command, args.split(' '), { cwd: this.options.rootDir }, { env: process.env }) | ||
const r = spawn.sync(command, args.split(' '), { cwd: this.options.rootDir, env: process.env }) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cross-spawn
doesn't take a third parameter
Thanks @danielroe for improvements But i think having tsdocs for internal build scripts is unnecessary complexity until we migrate repo to typescript for nuxt3 |
Types of changes
Description
This fixes a couple of minor issues (see GitHub comments).
Checklist: