Skip to content

Commit f0894d5

Browse files
authored
Merge pull request #132 from github/dont-send-matrix-vars
Don't send `matrix_vars` field in status reports.
2 parents 1ef33b0 + 1fb3aaf commit f0894d5

File tree

3 files changed

+3
-5
lines changed

3 files changed

+3
-5
lines changed

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: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -146,8 +146,6 @@ export interface StatusReportBase {
146146
"job_name": string;
147147
// Analysis key, normally composed from the workflow path and job name
148148
"analysis_key": string;
149-
// Value of the matrix for this instantiation of the job
150-
"matrix_vars"?: string;
151149
// Commit oid that the workflow was triggered on
152150
"commit_oid": string;
153151
// Ref that the workflow was triggered on
@@ -229,7 +227,7 @@ export async function createStatusReportBase(
229227
}
230228
let matrix: string | undefined = core.getInput('matrix');
231229
if (matrix) {
232-
statusReport.matrix_vars = matrix;
230+
// Temporarily do nothing.
233231
}
234232

235233
return statusReport;

0 commit comments

Comments
 (0)