Skip to content

Commit ca381ca

Browse files
authored
fix: Remove artifactURI usage from SARIF output (#141)
Signed-off-by: Eddie Knight <[email protected]>
1 parent ad73a8b commit ca381ca

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

pluginkit/evaluation_orchestrator.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -221,9 +221,8 @@ func (v *EvaluationOrchestrator) WriteResults() error {
221221
// Use "README.md" as artifactURI for repository-level assessments
222222
// GitHub Code Scanning requires PhysicalLocation, and "README.md" is a common file path
223223
// that satisfies this requirement (as recommended in gemara's ToSARIF documentation)
224-
artifactURI := "README.md"
225224
for _, suite := range v.Evaluation_Suites {
226-
sarifBytes, sarifErr := suite.EvaluationLog.ToSARIF(artifactURI, suite.catalog)
225+
sarifBytes, sarifErr := suite.EvaluationLog.ToSARIF("", suite.catalog)
227226
if sarifErr != nil {
228227
err = errMod(sarifErr, "wr25")
229228
break

0 commit comments

Comments
 (0)