diff --git a/.yarn/versions/30ae755e.yml b/.yarn/versions/30ae755e.yml new file mode 100644 index 000000000000..c211beb49726 --- /dev/null +++ b/.yarn/versions/30ae755e.yml @@ -0,0 +1,24 @@ +releases: + "@yarnpkg/cli": patch + "@yarnpkg/plugin-workspace-tools": patch + +declined: + - "@yarnpkg/plugin-compat" + - "@yarnpkg/plugin-constraints" + - "@yarnpkg/plugin-dlx" + - "@yarnpkg/plugin-essentials" + - "@yarnpkg/plugin-init" + - "@yarnpkg/plugin-interactive-tools" + - "@yarnpkg/plugin-nm" + - "@yarnpkg/plugin-npm-cli" + - "@yarnpkg/plugin-pack" + - "@yarnpkg/plugin-patch" + - "@yarnpkg/plugin-pnp" + - "@yarnpkg/plugin-pnpm" + - "@yarnpkg/plugin-stage" + - "@yarnpkg/plugin-typescript" + - "@yarnpkg/plugin-version" + - "@yarnpkg/builder" + - "@yarnpkg/core" + - "@yarnpkg/doctor" + - "@yarnpkg/pnp" diff --git a/packages/plugin-workspace-tools/sources/commands/foreach.ts b/packages/plugin-workspace-tools/sources/commands/foreach.ts index 47668fc447c8..b63e9e54c518 100644 --- a/packages/plugin-workspace-tools/sources/commands/foreach.ts +++ b/packages/plugin-workspace-tools/sources/commands/foreach.ts @@ -405,7 +405,7 @@ export default class WorkspacesForeachCommand extends BaseCommand { needsProcessing.delete(identHash); processing.delete(workspace.anchoredDescriptor.descriptorHash); - return exitCode; + return {workspace, exitCode}; })); // If we're not executing processes in parallel we can just wait for it @@ -424,13 +424,15 @@ export default class WorkspacesForeachCommand extends BaseCommand { return; } - const exitCodes: Array = await Promise.all(commandPromises); - const errorCode = exitCodes.find(code => code !== 0); + const results: Array<{ workspace: Workspace, exitCode: number }> = await Promise.all(commandPromises); + results.forEach(({workspace, exitCode}) => { + if (exitCode !== 0) { + report.reportError(MessageName.UNNAMED, `The command failed in workspace ${structUtils.prettyLocator(configuration, workspace.anchoredLocator)} with exit code ${exitCode}`); + } + }); - // The order in which the exit codes will be processed is fairly - // opaque, so better just return a generic "1" for determinism. - if (finalExitCode === null) - finalExitCode = typeof errorCode !== `undefined` ? 1 : finalExitCode; + const exitCodes = results.map(result => result.exitCode); + const errorCode = exitCodes.find(code => code !== 0); if ((this.topological || this.topologicalDev) && typeof errorCode !== `undefined`) { report.reportError(MessageName.UNNAMED, `The command failed for workspaces that are depended upon by other workspaces; can't satisfy the dependency graph`);