Skip to content

improve: set sonar sonar version explicitly#3501

Closed
csviri wants to merge 2 commits into
operator-framework:mainfrom
csviri:explicit-sonar-version
Closed

improve: set sonar sonar version explicitly#3501
csviri wants to merge 2 commits into
operator-framework:mainfrom
csviri:explicit-sonar-version

Conversation

@csviri

@csviri csviri commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Set sonar version to 5.7.0.6970 in GitHub Action

Signed-off-by: Attila Mészáros a_meszaros@apple.com

Set sonar version to 5.7.0.6970 in GitHub Action

Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Copilot AI review requested due to automatic review settings July 16, 2026 10:12
@openshift-ci
openshift-ci Bot requested review from metacosm and xstefank July 16, 2026 10:12

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Pins the Sonar Maven plugin version in the Sonar GitHub Actions workflow to make Sonar analysis execution deterministic in CI.

Changes:

  • Pin org.sonarsource.scanner.maven:sonar-maven-plugin to 5.7.0.6970 in the Sonar workflow Maven invocation.

Comment thread .github/workflows/sonar.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

@xstefank xstefank left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

however, maybe this should be in pom, not hardcoded in CI.

@csviri

csviri commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator Author

Sorry just started to experiment with this, actually will close this for now, since the core of the sonar issues is somewhere else that we will have to tackle in a separate PR anyways.

@csviri csviri closed this Jul 20, 2026
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