For some reason showing SARIF report results stopped working kotlinlang #detekt

Channels

fluid-libraries

jhipster-kotlin

effectivekotlin

effective-kotlin

android-architecture

algolialibraries

stayhungrystayfoolish

pattern-matching

kotlin-data-storage

skia-wasm-interop-temp

squarelibraries

beepiz-libraries

cscenter-course-2016

kotlin-foundation

compose-android

lambdaworld_cadiz

kotlin-pakistan

java-to-kotlin-refactoring

event21-community-content

fb-internal-demo

kotlinprogrammers

kotlinx-datetime

framework-elide

github-workflows-kt

library-development

arrow-contributors

spring-security

pocketgitclient

domain-driven-design

swiss-user-group

refactoring-to-kotlin

ohio-kotlin-users

introduce-yourself

kotlin-by-example

kotlin-multiplatform-contest

graviton-browser

tricity-kotlin-user-group

kotlin-in-action

android-databinding

kotlintest-devs

cincinnati-user-group

codemash-precompiler

kotlingforbeginners

prozis-android-backup

realworldkotlin

connect-audit-events

russian-kotlinasfirst

androidgithubprojects

intellij-tricks

machinelearningbawas

swagger-gradle-codegen

100daysofkotlin

competitive-programming

compose-ui-showcase

androidx-xprocessing

navigation-architecture-component

strife-discord-lib

embedded-kotlin

image-processing

myndocs-oauth2-server

leedskotlinusergroup

kotest-contributors

practical-functional-programming

kotlinultimatechallenge

k2-early-adopters

kotlinforbeginners

compose-desktop

competitivecoding

100daysofkotlin-2021

refreshversions

language-evolution

jvm-ir-backend-feedback

python-contributors

codingconventions

sofl-user-group

getting-started

language-proposals

intellij-plugins

Title

s

Sebastian Schuberth

09/15/2022, 12:37 PM

For some reason showing SARIF report results stopped working for me, although the report seems to be uploaded correctly. Any idea why might go wrong in this test PR? The GitHub action is configured like so: https://github.com/oss-review-toolkit/ort/blob/26392e163d2dd63dd9937f5369e458315c39cd08/.github/workflows/static-analysis.yml#L24-L32

Hmm, running

detekt

locally confirms that the violation is present in

analyzer/build/reports/detekt/detekt.sarif

, but not in

build/reports/detekt/detekt.sarif

. So maybe something does wring with merging the reports?

Hmm, I thought as of detekt 1.19.0 SARIF reports are auto-merged, and there is no need to manually create a merge task anymore. Is that not the case?

Ok, reintroducing the manual merge of reports fixes it.

10 Views