Skip to content

SC-45749 Migrate to dateless license headers#286

Merged
claire-villard-sonarsource merged 1 commit intomasterfrom
claire/SC-45749/headers
Apr 10, 2026
Merged

SC-45749 Migrate to dateless license headers#286
claire-villard-sonarsource merged 1 commit intomasterfrom
claire/SC-45749/headers

Conversation

@claire-villard-sonarsource
Copy link
Copy Markdown
Contributor

Please be aware that we are not actively looking for feature contributions. The truth is that it's extremely difficult for someone outside SonarSource to comply with our roadmap and expectations. Therefore, we typically only accept minor cosmetic changes and typo fixes. If you would like to see a new feature, please create a new thread in the forum "Suggest new features".

With that in mind, if you would like to submit a code contribution, make sure that you adhere to the following guidelines and all tests are passing:

  • Please explain your motives to contribute this change: what problem you are trying to fix, what improvement you are trying to make
  • Use the following formatting style: SonarSource/sonar-developer-toolset
  • Provide a unit test for any code you changed
  • If there is a JIRA ticket available, please make your commits and pull request start with the ticket ID (SCANJLIB-XXXX)

We will try to give you feedback on your contribution as quickly as possible.

Thank You!
The SonarSource Team

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

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

SC-45749

@sonar-review-alpha
Copy link
Copy Markdown

sonar-review-alpha bot commented Apr 10, 2026

Summary

This PR removes date ranges from all copyright headers across the project and standardizes them to a dateless format. Specifically:

  • Copyright headers like "Copyright (C) 2011-2025 SonarSource Sàrl" become "Copyright (C) SonarSource Sàrl"
  • README.md and NOTICE.txt are updated to match the new standardized format
  • The change is purely mechanical—all 125 files are updated by removing the year range, affecting no code logic or functionality

This aligns copyright headers with modern best practices where copyright dates are not maintained inline.

What reviewers should know

Scope: 125 files changed, all with identical 2-line modifications (removing date ranges).

Files to spot-check:

  • NOTICE.txt and README.md for the header format change
  • One file from each module (batch-interface, cache, main lib, tests, integration tests) to verify consistency

Pattern: All changes follow the same pattern—the only variation is between different header styles (Java block comments, Maven pom.xml, etc.), but the date removal is consistent everywhere.

No functional impact: This is purely a header metadata update. No imports, logic, dependencies, or test behavior are affected. The stat diff of "+1, -1" on every file confirms only header lines were touched.


  • Generate Walkthrough
  • Generate Diagram

🗣️ Give feedback

sonar-review-alpha[bot]

This comment was marked as outdated.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

There's the license in README.md to change

@claire-villard-sonarsource
Copy link
Copy Markdown
Contributor Author

There's the license in README.md to change

Good catch! Done

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, purely mechanical change — no logic, no behavior, no risk. Every one of the 125 modified files has exactly one line changed: the copyright year range stripped out, leaving only the copyright holder name. The README.md change also standardizes the phrasing to match the rest of the codebase ("Copyright (C) SonarSource Sàrl" instead of the previous "Copyright 2011-2025 SonarSource.").

🗣️ Give feedback

@sonarqube-next
Copy link
Copy Markdown

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@claire-villard-sonarsource claire-villard-sonarsource merged commit f4347b4 into master Apr 10, 2026
10 checks passed
@claire-villard-sonarsource claire-villard-sonarsource deleted the claire/SC-45749/headers branch April 10, 2026 12:24
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