Skip to content

SCANPY-238 Update license headers#302

Merged
marc-jasper-sonarsource merged 5 commits intomasterfrom
MJ/SCANPY-238
Mar 24, 2026
Merged

SCANPY-238 Update license headers#302
marc-jasper-sonarsource merged 5 commits intomasterfrom
MJ/SCANPY-238

Conversation

@marc-jasper-sonarsource
Copy link
Copy Markdown
Contributor

No description provided.

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

hashicorp-vault-sonar-prod bot commented Mar 19, 2026

SCANPY-238

@sonar-review-alpha
Copy link
Copy Markdown

sonar-review-alpha bot commented Mar 19, 2026

Summary

Update all license headers to reflect new copyright owner and extended date range. Changed copyright owner from 'SonarSource SA.' to 'SonarSource Sàrl' and extended the copyright year range from '2011-2024' to '2011-2026'. Also removed the trailing period from the company name in the license header template.

What reviewers should know

This is a bulk administrative change touching nearly 40 files. The changes are systematic and mechanical:

  • license_header.tmpl: Template source — removed trailing period from ${owner}
  • scripts/generate_licenseheaders.sh: License generation script — updated owner parameter and year range from the script that auto-generates headers
  • All other files: Python source files across src/, tests/, and tests/its/ — reflect the template changes

Expect identical changes across all files: copyright owner name, date range, and removal of the period. No functional code changes. Use a simple text search to spot-check one or two files if desired, then trust the mechanical application of the rest.


  • Generate Walkthrough
  • Generate Diagram

🗣️ Give feedback

@sonar-review-alpha
Copy link
Copy Markdown

sonar-review-alpha bot commented Mar 19, 2026

Summary

Updates all license headers project-wide to reflect a change in copyright owner from "SonarSource SA" to "SonarSource Sàrl" and extends the copyright year range to 2026. Also removes the trailing period from the owner name. These are cosmetic, administrative-only changes with no functional impact on code behavior.

What reviewers should know

Where to start: Review license_header.tmpl and scripts/generate_licenseheaders.sh first—these define the authoritative header format and drive all the individual file updates.

What changed: The copyright line in headers across all Python source and test files now reads Copyright (C) 2011-2026 SonarSource Sàrl (instead of 2011-2024 SonarSource SA.). Note the accent in "Sàrl" and removal of the trailing period.

No functional changes: This is purely administrative—no code logic, behavior, or tests were modified. All updates are consistent across ~30+ Python files in src/, tests/, and integration test directories.

Verification approach: Spot-check a few files across different directories to confirm the pattern is applied consistently; checking the template and script is sufficient to understand the entire change.


  • Generate Walkthrough
  • Generate Diagram

🗣️ Give feedback

@sonar-review-alpha
Copy link
Copy Markdown

sonar-review-alpha bot commented Mar 19, 2026

Summary

Updates all license headers across the project to reflect organizational and date changes. Changes company name from 'SonarSource SA' to 'SonarSource Sàrl', updates copyright year range from 2011-2024 to 2011-2026, and removes trailing period from the license template. Updates build pipeline, documentation, and license generation scripts to use the new values.

What reviewers should know

This is a straightforward automated update of license headers. All changes are identical across the codebase:

  1. Header template (license_header.tmpl): Minor formatting change — removes period after owner name
  2. Source/test files: ~40 files with copyright header updates in src/, tests/, and tests/its/ directories
  3. Automation: Build workflow (.github/workflows/build.yml), contributing guide (CONTRIBUTING.md), and generation script (scripts/generate_licenseheaders.sh) all reference the updated company name and year

No logic changes. Verify the changes are consistent across all files (search for any remaining '2011-2024' or 'SonarSource SA' references if concerned about completeness).


  • 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.

Conclusion

Clean, purely mechanical update with no logic or functional changes — safe to approve.

🗣️ 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.

Conclusion: Clean, mechanical update. All 40+ files are consistently updated, the template and scripts are aligned, and the CI check enforces the new header going forward.

🗣️ Give feedback

Copy link
Copy Markdown
Contributor

@thomas-serre-sonarsource thomas-serre-sonarsource left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for doing this!!

I think there is still one file where you should do the change CONTRIBUTING.md :

## License header

Before pushing, please check if all files have a license header.
If not all files have a license header please execute: 

poetry run licenseheaders -t license_header.tmpl -o "SonarSource SA" -y 2011-2024 -n "Sonar Scanner Python" -E .py -d src/
poetry run licenseheaders -t license_header.tmpl -o "SonarSource SA" -y 2011-2024 -n "Sonar Scanner Python" -E .py -d tests/

@sonarqube-next
Copy link
Copy Markdown

Quality Gate passed Quality Gate passed for 'Python Scanner'

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

Copy link
Copy Markdown
Contributor

@thomas-serre-sonarsource thomas-serre-sonarsource left a comment

Choose a reason for hiding this comment

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

LGTM!

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! ✅

🗣️ Give feedback

@marc-jasper-sonarsource marc-jasper-sonarsource merged commit fdb3129 into master Mar 24, 2026
21 checks passed
@marc-jasper-sonarsource marc-jasper-sonarsource deleted the MJ/SCANPY-238 branch March 24, 2026 10:09
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