Skip to content

SONARSCALA-64 Use "sonar.scanner.skipJreProvisioning" in integration tests#56

Merged
alban-auzeill merged 1 commit intomasterfrom
godin/skipJreProvisioning
Jun 10, 2025
Merged

SONARSCALA-64 Use "sonar.scanner.skipJreProvisioning" in integration tests#56
alban-auzeill merged 1 commit intomasterfrom
godin/skipJreProvisioning

Conversation

@Godin
Copy link
Member

@Godin Godin commented Jun 2, 2025

SONARSCALA-64

On ephemeral CI machine this avoids unnecessary downloading and unpacking of JRE from SQ and thus reduces time of execution of the first project analysis in integration tests.

During execution of integration tests we already have suitable JDK.

Testing of JRE provisioning feature should not be the responsibility of analyzers.

On ephemeral CI machine this avoids unnecessary downloading and
unpacking of JRE from SQ and thus reduces time of execution of the
first project analysis in integration tests.

During execution of integration tests we already have suitable JDK.

Testing of JRE provisioning feature should not be the responsibility
of analyzers.
@Godin Godin self-assigned this Jun 2, 2025
@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod bot changed the title Use "sonar.scanner.skipJreProvisioning" in integration tests SONARSCALA-64 Use "sonar.scanner.skipJreProvisioning" in integration tests Jun 2, 2025
@sonarqube-next
Copy link

sonarqube-next bot commented Jun 2, 2025

@Godin Godin marked this pull request as ready for review June 2, 2025 12:25
@Godin Godin requested a review from a team June 2, 2025 12:25
@github-actions
Copy link
Contributor

This PR is stale because it has been open 7 days with no activity. If there is no activity in the next 7 days it will be closed automatically

@github-actions github-actions bot added the stale label Jun 10, 2025
@alban-auzeill alban-auzeill merged commit f08620d into master Jun 10, 2025
10 checks passed
@alban-auzeill alban-auzeill deleted the godin/skipJreProvisioning branch June 10, 2025 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants