Skip to content

JS-1605 Fix licenses#46

Merged
zglicz merged 1 commit intomasterfrom
fix-licenses
Apr 16, 2026
Merged

JS-1605 Fix licenses#46
zglicz merged 1 commit intomasterfrom
fix-licenses

Conversation

@zglicz
Copy link
Copy Markdown
Contributor

@zglicz zglicz commented Apr 16, 2026

Part of

@zglicz zglicz requested a review from a team as a code owner April 16, 2026 07:31
@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod bot changed the title Fix licenses JS-1605 Fix licenses Apr 16, 2026
@hashicorp-vault-sonar-prod
Copy link
Copy Markdown

hashicorp-vault-sonar-prod bot commented Apr 16, 2026

JS-1605

@sonar-review-alpha
Copy link
Copy Markdown

sonar-review-alpha bot commented Apr 16, 2026

Summary

This PR corrects license headers across four Java files in the Maven plugin. The changes fix three issues:

  1. Copyright date: Removes the invalid year range "2025-2025" and keeps just "SonarSource Sàrl" without specific years
  2. License statement: Removes "This program is free software;" since this project uses a Source-Available license, not a free software license
  3. Entity name consistency: Changes "SonarSource SA" to "SonarSource Sàrl" to match the copyright holder name above it

The license statement is also slightly reworded for clarity but maintains the same legal meaning.

What reviewers should know

All four files receive identical changes to their license headers:

  • ChecksumException.java
  • CompressMojo.java
  • DownloadRuntimesMojo.java
  • RuntimeFlavor.java

Only the file headers are modified; no code logic is affected. These are purely documentation/license corrections.


  • Generate Walkthrough
  • Generate Diagram

🗣️ Give feedback

Copy link
Copy Markdown

@sonar-review-alpha sonar-review-alpha bot left a comment

Choose a reason for hiding this comment

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

LGTM! ✅

Clean, mechanical change — license headers are correctly updated across all four files with no functional impact.

🗣️ Give feedback

@sonarqube-next
Copy link
Copy Markdown

Quality Gate passed Quality Gate passed for 'NodeJS Maven Plugin'

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube

@zglicz zglicz merged commit c874e3c into master Apr 16, 2026
6 checks passed
@zglicz zglicz deleted the fix-licenses branch April 16, 2026 07:39
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.

1 participant