Skip to content

Commit 930b240

Browse files
Copilotdmichon-msft
andcommitted
Update OperationExecutionRecord to only set logFilePaths conditionally
Co-authored-by: dmichon-msft <26827560+dmichon-msft@users.noreply.github.com> Agent-Logs-Url: https://github.com/microsoft/rushstack/sessions/32548397-4715-49df-ac93-946ff98db320
1 parent ded4068 commit 930b240

2 files changed

Lines changed: 13 additions & 1 deletion

File tree

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
{
2+
"changes": [
3+
{
4+
"packageName": "@microsoft/rush",
5+
"comment": "Update OperationExecutionRecord to only set `this.logFilePaths` if the value to be assigned is not `undefined`",
6+
"type": "patch"
7+
}
8+
],
9+
"packageName": "@microsoft/rush"
10+
}

libraries/rush-lib/src/logic/operations/OperationExecutionRecord.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -307,7 +307,9 @@ export class OperationExecutionRecord implements IOperationRunnerContext, IOpera
307307
logFilenameIdentifier: `${this._operationMetadataManager.logFilenameIdentifier}${logFileSuffix}`
308308
})
309309
: undefined;
310-
this.logFilePaths = logFilePaths;
310+
if (logFilePaths !== undefined) {
311+
this.logFilePaths = logFilePaths;
312+
}
311313

312314
const projectLogWritable: TerminalWritable | undefined = logFilePaths
313315
? await initializeProjectLogFilesAsync({

0 commit comments

Comments
 (0)