Skip to content

SONARJAVA-6536 Make sonar-java required for Kotlin#5711

Merged
petertrr merged 1 commit into
masterfrom
petertrr/SONARKT-765
Jun 29, 2026
Merged

SONARJAVA-6536 Make sonar-java required for Kotlin#5711
petertrr merged 1 commit into
masterfrom
petertrr/SONARKT-765

Conversation

@petertrr

@petertrr petertrr commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Part of SONARKT-765


Summary by Gitar

  • Build configuration:
    • Updated requiredForLanguages in sonar-java-plugin/pom.xml to include kotlin as a required language.

This will update automatically on new commits.

@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod Bot changed the title Make sonar-java required for Kotlin SONARJAVA-6536 Make sonar-java required for Kotlin Jun 29, 2026
@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

SONARJAVA-6536

@gitar-bot

gitar-bot Bot commented Jun 29, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

Updates requiredForLanguages in the Maven configuration to include Kotlin, ensuring sonar-java is correctly identified as a dependency. No issues found.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@sonarqube-next

Copy link
Copy Markdown

@petertrr petertrr marked this pull request as ready for review June 29, 2026 13:46

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@petertrr petertrr merged commit 32e4f07 into master Jun 29, 2026
17 checks passed
@petertrr petertrr deleted the petertrr/SONARKT-765 branch June 29, 2026 14:23
Comment thread sonar-java-plugin/pom.xml
<skipDependenciesPackaging>true</skipDependenciesPackaging>
<pluginClass>org.sonar.plugins.java.JavaPlugin</pluginClass>
<requiredForLanguages>java,jsp</requiredForLanguages>
<requiredForLanguages>java,jsp,kotlin</requiredForLanguages>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@petertrr maybe also add explanatory comment?

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