Skip to content

CORE=69: bump sonar plugin to gain JRE auto-provisioning - #141

Merged
davidangb merged 4 commits into
mainfrom
da_CORE-69_bumpSonarPlugin
Sep 2, 2026
Merged

davidangb merged 4 commits into
mainfrom
da_CORE-69_bumpSonarPlugin

Conversation

@davidangb

@davidangb davidangb commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Bumps the Sonar gradle plugin from 4.4.1.3373 to 7.4.0.8496, and moves the SourceClear plugin to the plugins {} block.

Sonar scans are failing (example) with an error:

The version of Java (17) used to run this analysis is deprecated, and SonarQube Cloud no longer supports it. Please upgrade to Java 21 or later.

The Sonar plugin, as of 6.0, downloads its own dedicated JRE. We were on an older plugin version that didn't know how to do this. So: upgrade the plugin.

However, upgrading the Sonar plugin ran into class-conflict errors with a SourceClear transitive dependency. The SourceClear plugin didn't need to be declared in buildSrc, so move it to plugins {...} so gradle can better manage its classpath.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Benchmark

Details
Benchmark suite Current: eaa7710 Previous: 20035c2 Ratio
bio.terra.pfb.LibraryBenchmarks.showNodesMedium 1477.8645856104768 ops/s 1486.1310243076252 ops/s 0.99
bio.terra.pfb.LibraryBenchmarks.showNodesSmall 24304.34331956949 ops/s 21446.130770847907 ops/s 1.13
bio.terra.pfb.PfbReaderBenchmarks.convertEnum 7084710.503332313 ops/s 5178382.502453218 ops/s 1.37

This comment was automatically generated by workflow using github-action-benchmark.

@sonarqubecloud

sonarqubecloud Bot commented Sep 2, 2026

Copy link
Copy Markdown

@sonarqubecloud

sonarqubecloud Bot commented Sep 2, 2026

Copy link
Copy Markdown

@davidangb
davidangb marked this pull request as ready for review September 2, 2026 13:31
@davidangb
davidangb requested a review from a team as a code owner September 2, 2026 13:32
@davidangb
davidangb requested a review from snf2ye September 2, 2026 13:32
@davidangb
davidangb merged commit cdf0600 into main Sep 2, 2026
11 checks passed
@davidangb
davidangb deleted the da_CORE-69_bumpSonarPlugin branch September 2, 2026 14:05
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