Skip to content

Commit 631929a

Browse files
authored
Merge pull request #135 from github/revert-dont-send-matrix-vars
revert: 'Don't send `matrix_vars` field in status reports.'
2 parents 69bf3f2 + 128c2cf commit 631929a

3 files changed

Lines changed: 5 additions & 3 deletions

File tree

lib/util.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lib/util.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/util.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,8 @@ export interface StatusReportBase {
166166
"job_name": string;
167167
// Analysis key, normally composed from the workflow path and job name
168168
"analysis_key": string;
169+
// Value of the matrix for this instantiation of the job
170+
"matrix_vars"?: string;
169171
// Commit oid that the workflow was triggered on
170172
"commit_oid": string;
171173
// Ref that the workflow was triggered on
@@ -247,7 +249,7 @@ export async function createStatusReportBase(
247249
}
248250
let matrix: string | undefined = core.getInput('matrix');
249251
if (matrix) {
250-
// Temporarily do nothing.
252+
statusReport.matrix_vars = matrix;
251253
}
252254

253255
return statusReport;

0 commit comments

Comments
 (0)