Remove year from license headers - #500
Conversation
Signed-off-by: Felix Riegger <felix.riegger@sap.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (53)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe change removes fixed year values from SPDX copyright notices in ChangesLicense header migration
Priority: ⬇️ Low — Defer the license-header cleanup because it is a low-complexity, comment-only change with no functional product impact. Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This change makes copyright notices year-free while preserving SPDX license markers and does not alter application behavior. No current merge-readiness risk remains. Suggested reviewers: 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (3 passed)
Full details: Linked Issues checkExplanation The pull request updates many existing SPDX copyright headers and REUSE.toml, which addresses the main issue objective. The provided context does not confirm that all tracked files were checked, that every license marker remains valid, that a safe rewrite process was added or documented when needed, or that make check-license passes.
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
Fixes #499
Summary by CodeRabbit
Documentation
Chores