Skip to content

SONARJAVA-6182 Switch Guava project used in ruling tests to Java 21#59

Merged
asya-vorobeva merged 2 commits intomasterfrom
asya/fix-guava-tests
Mar 2, 2026
Merged

SONARJAVA-6182 Switch Guava project used in ruling tests to Java 21#59
asya-vorobeva merged 2 commits intomasterfrom
asya/fix-guava-tests

Conversation

@asya-vorobeva
Copy link
Contributor

@asya-vorobeva asya-vorobeva commented Mar 2, 2026

@asya-vorobeva asya-vorobeva marked this pull request as draft March 2, 2026 09:28
@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod bot changed the title Switch Guava project used in ruling tests to Java 17 JAVASE-166 Switch Guava project used in ruling tests to Java 17 Mar 2, 2026
@asya-vorobeva asya-vorobeva force-pushed the asya/fix-guava-tests branch from e5f1e19 to d4a12f6 Compare March 2, 2026 12:49
@asya-vorobeva asya-vorobeva force-pushed the asya/fix-guava-tests branch from d4a12f6 to 463d35d Compare March 2, 2026 12:50
@asya-vorobeva asya-vorobeva changed the title JAVASE-166 Switch Guava project used in ruling tests to Java 17 SONARJAVA-6182 Switch Guava project used in ruling tests to Java 21 Mar 2, 2026
@asya-vorobeva asya-vorobeva marked this pull request as ready for review March 2, 2026 13:17
with:
working-directory: its/ruling
version: 2025.7.12
- name: Override Java version for specific profiles
Copy link
Contributor

Choose a reason for hiding this comment

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

Java 21 is the default in mise.toml and we do not override anything (no java_version any more), so I think we can remove this step completely. (And I have not idea why 17 worked when we do not have it in mise.toml).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We've had it in internal its/ruling mise config file (which I remove here).

@sonarqube-next
Copy link

sonarqube-next bot commented Mar 2, 2026

@asya-vorobeva asya-vorobeva merged commit 23b6ee0 into master Mar 2, 2026
6 checks passed
@asya-vorobeva asya-vorobeva deleted the asya/fix-guava-tests branch March 2, 2026 16:35
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.

2 participants