Skip to content

Enable retrieving sonarqube scan results for PRs#784

Merged
AlexKantor87 merged 5 commits intomainfrom
get-sonar-scan-for-pr
Apr 15, 2026
Merged

Enable retrieving sonarqube scan results for PRs#784
AlexKantor87 merged 5 commits intomainfrom
get-sonar-scan-for-pr

Conversation

@FayeSGW
Copy link
Copy Markdown
Contributor

@FayeSGW FayeSGW commented Apr 15, 2026

This PR allows SonarQube scans for pull-requests to be attested to Kosli.
This can be done through SonarQube's metadata file (which requires no extra work for the user) or through providing a --pull-request flag.
It is also possible to provide a --sonar-ce-task-url flag, which bypasses the need for the metadata file.

@claude
Copy link
Copy Markdown
Contributor

claude bot commented Apr 15, 2026

Claude encountered an error —— View job


Reviewing PR #784

  • Run static analysis (build, vet, mod tidy)
  • Read changed files and gather context
  • Review code quality, bugs, edge cases, and tests
  • Post inline comments and summary

Comment thread cmd/kosli/attestSonar.go Outdated
Comment thread internal/sonar/sonar.go Outdated
Comment thread internal/sonar/sonar.go Outdated
Comment thread internal/sonar/sonar.go Outdated
Comment thread internal/sonar/sonar.go Outdated
Comment thread internal/sonar/sonar.go
FayeSGW and others added 3 commits April 15, 2026 14:32
Merge both --pull-request and --sonar-ce-task-url flags, update long
description to document method 2 supporting PR scans, add key+PR
example, fix test golden output, and renumber tests.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Comment thread internal/sonar/sonar.go
Comment thread internal/sonar/sonar.go
Comment thread internal/sonar/sonar.go Outdated
Comment thread internal/sonar/sonar.go
Comment thread internal/sonar/sonar.go
Comment thread internal/sonar/sonar.go
Comment thread internal/sonar/sonar.go
Comment thread cmd/kosli/attestSonar_test.go
@AlexKantor87 AlexKantor87 merged commit 6239887 into main Apr 15, 2026
18 of 19 checks passed
@AlexKantor87 AlexKantor87 deleted the get-sonar-scan-for-pr branch April 15, 2026 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants