Skip to content

SONARJAVA-6016 Migrate the Guava project in the ruling tests to Java 21#5459

Merged
aurelien-coet-sonarsource merged 3 commits intomasterfrom
ac/SONARJAVA-6016
Feb 17, 2026
Merged

SONARJAVA-6016 Migrate the Guava project in the ruling tests to Java 21#5459
aurelien-coet-sonarsource merged 3 commits intomasterfrom
ac/SONARJAVA-6016

Conversation

@aurelien-coet-sonarsource
Copy link
Copy Markdown
Contributor

@aurelien-coet-sonarsource aurelien-coet-sonarsource commented Feb 13, 2026

See Related PR in the ruling_java project.

@hashicorp-vault-sonar-prod
Copy link
Copy Markdown
Contributor

hashicorp-vault-sonar-prod Bot commented Feb 13, 2026

SONARJAVA-6016

Copy link
Copy Markdown
Contributor

@asya-vorobeva asya-vorobeva left a comment

Choose a reason for hiding this comment

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

  • Add in PR description reference to the related one SonarSource/ruling_java#9
  • About changes in its/sources: I'm sure that we don't need to change everything here, cause it's a git submodule linked to ruling_java repo, should be updated automatically after merging the first PR.

@aurelien-coet-sonarsource
Copy link
Copy Markdown
Contributor Author

Yes, once the PR in ruling_java will be merged, we'll just need to update the submodule to point to the latest commit on master instead of the one on my branch.

@asya-vorobeva
Copy link
Copy Markdown
Contributor

Yes, once the PR in ruling_java will be merged, we'll just need to update the submodule to point to the latest commit on master instead of the one on my branch.

It means you can revert changes in its\sources. Let's do it after merge that PR(already approved)

@aurelien-coet-sonarsource
Copy link
Copy Markdown
Contributor Author

SONARJAVA-6016 Update the Guava project to support Java 21 has been merged and the its/sources submodule now points to the latest commit on master.

Copy link
Copy Markdown
Contributor

@asya-vorobeva asya-vorobeva left a comment

Choose a reason for hiding this comment

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

💯

Comment thread .github/workflows/build.yml Outdated
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.

We don't need this logic any more, so we can inline LATEST_RELEASE below.

@sonarqube-next
Copy link
Copy Markdown

@aurelien-coet-sonarsource aurelien-coet-sonarsource merged commit abad05a into master Feb 17, 2026
14 checks passed
@aurelien-coet-sonarsource aurelien-coet-sonarsource deleted the ac/SONARJAVA-6016 branch February 17, 2026 15:45
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