Merge pull request #65 from smartcontractkit/fix/BCF-3321-fix-gh-work…

…flows

Update linter to detect unused vars and sonar workflow to propagate linter issues

.github/workflows/sonar-scan.yml

@@ -47,6 +47,7 @@ jobs:
         with:
           workflow: ci-lint.yml
           name: golangci-lint-report
+          workflow_conclusion: ""
           if_no_artifact_found: warn
 
       - name: Download go unit test coverage reports
@@ -55,6 +56,7 @@ jobs:
         with:
           workflow: ci-test.yml
           name: go-test-results
+          workflow_conclusion: ""
           if_no_artifact_found: warn
 
       - name: Set SonarQube Report Paths

.golangci.yml

@@ -12,7 +12,6 @@ linters:
     # These are all considered deprecated: https://github.com/golangci/golangci-lint/issues/1841
     - deadcode
     - structcheck
-    - unused
     - varcheck
 linters-settings:
   exhaustive: