Skip to content

Remove year from license headers - #144

Merged
afritzler merged 1 commit into
mainfrom
chore/143-remove-year-license-headers
Sep 9, 2026
Merged

Remove year from license headers#144
afritzler merged 1 commit into
mainfrom
chore/143-remove-year-license-headers

Conversation

@friegger

@friegger friegger commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Fixes #143

Summary by CodeRabbit

  • Documentation
    • Updated copyright attribution text in the project’s licensing metadata.

Signed-off-by: Felix Riegger <felix.riegger@sap.com>
@friegger
friegger requested a review from a team as a code owner September 9, 2026 09:00
@coderabbitai

coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 2cc046d2-769f-465b-93b0-a83e8bf61827

📥 Commits

Reviewing files that changed from the base of the PR and between 2a8a0fa and fcf6905.

📒 Files selected for processing (1)
  • REUSE.toml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The change removes the leading year from both SPDX-FileCopyrightText values in REUSE.toml.

Changes

License header metadata

Layer / File(s) Summary
Update REUSE.toml annotations
REUSE.toml
Both annotation blocks now use year-free SPDX-FileCopyrightText values.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to fcf69

This updates REUSE metadata to use year-free copyright text without changing license identifiers or runtime behavior. No current merge-readiness risk remains.

Suggested reviewers: afritzler

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description includes the issue reference but omits the required Proposed Changes section and its summary bullets. Add a Proposed Changes section that describes the REUSE.toml updates and confirms the intended license-header changes. Keep the Fixes #143 reference.
Linked Issues check ⚠️ Warning The change updates REUSE.toml, which is in scope, but it does not address the linked issue's broader requirement to update all tracked files, preserve valid SPDX markers, and validate the migration wi… Update all applicable tracked files with year-free SPDX copyright text, preserve valid SPDX license markers, and provide evidence that make check-license passes. Add or document a safe rewrite process if required.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the removal of years from license headers.
Out of Scope Changes check ✅ Passed The REUSE.toml changes are related to the linked issue and do not introduce unrelated changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Full details: Linked Issues check

Explanation

The change updates REUSE.toml, which is in scope, but it does not address the linked issue's broader requirement to update all tracked files, preserve valid SPDX markers, and validate the migration with make check-license.

  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/143-remove-year-license-headers

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@afritzler
afritzler merged commit ebbfd87 into main Sep 9, 2026
6 checks passed
@afritzler
afritzler deleted the chore/143-remove-year-license-headers branch September 9, 2026 09:06
@hardikdr hardikdr added this to Roadmap Sep 10, 2026
@github-project-automation github-project-automation Bot moved this to Done in Roadmap Sep 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Remove years from existing license headers

3 participants