Skip to content

Remove year from license headers - #500

Open
friegger wants to merge 1 commit into
mainfrom
chore/499-remove-year-license-headers
Open

Remove year from license headers#500
friegger wants to merge 1 commit into
mainfrom
chore/499-remove-year-license-headers

Conversation

@friegger

@friegger friegger commented Sep 8, 2026

Copy link
Copy Markdown

Fixes #499

Summary by CodeRabbit

  • Documentation

    • Updated SPDX copyright notices throughout the project to use year-independent attribution.
    • Standardized attribution for SAP SE, its affiliates, and IronCore contributors.
  • Chores

    • Refreshed copyright metadata in source files, tests, test data, mock packages, and project configuration.
    • No user-facing functionality or test behavior changed.

Signed-off-by: Felix Riegger <felix.riegger@sap.com>
@friegger
friegger requested review from a team as code owners September 8, 2026 13:44
@github-actions github-actions Bot added the chore label Sep 8, 2026
@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown

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: cdbcd926-4fe6-4fe6-8b7a-6ec2ae16b07e

📥 Commits

Reviewing files that changed from the base of the PR and between cdadc06 and 0e45f79.

📒 Files selected for processing (53)
  • REUSE.toml
  • buildutils/alias.go
  • buildutils/buildutils.go
  • buildutils/buildutils_suite_test.go
  • buildutils/buildutils_test.go
  • clientutils/clientutils.go
  • clientutils/clientutils_suite_test.go
  • clientutils/clientutils_test.go
  • clientutils/fieldindexer.go
  • clientutils/fieldindexer_test.go
  • clientutils/objectkey.go
  • clientutils/objectkey_test.go
  • clientutils/objectref.go
  • clientutils/objectref_test.go
  • cmdutils/switches/switches.go
  • cmdutils/switches/switches_suite_test.go
  • cmdutils/switches/switches_test.go
  • conditionutils/alias.go
  • conditionutils/alias_test.go
  • conditionutils/conditionutils.go
  • conditionutils/conditionutils_suite_test.go
  • conditionutils/conditionutils_test.go
  • configutils/configutils.go
  • configutils/configutils_suite_test.go
  • configutils/configutils_test.go
  • envtestutils/envtestutils.go
  • envtestutils/envtestutils_suite_test.go
  • envtestutils/envtestutils_test.go
  • kustomizeutils/kustomizeutils.go
  • kustomizeutils/kustomizeutils_suite_test.go
  • kustomizeutils/kustomizeutils_test.go
  • metautils/metautils.go
  • metautils/metautils_suite_test.go
  • metautils/metautils_test.go
  • migration/migration.go
  • migration/migration_test.go
  • mock/controller-runtime/client/doc.go
  • mock/controller-utils/clientutils/doc.go
  • mock/controller-utils/metautils/doc.go
  • modutils/alias.go
  • modutils/modutils.go
  • modutils/modutils_suite_test.go
  • modutils/modutils_test.go
  • testdata/testdata.go
  • testdata/testmod2/main.go
  • testdata/testmod2/submain/main.go
  • testutils/matchers/matchers.go
  • testutils/matchers/matchers_suite_test.go
  • testutils/matchers/matchers_test.go
  • testutils/testutils.go
  • unstructuredutils/unstructuredutils.go
  • unstructuredutils/unstructuredutils_suite_test.go
  • unstructuredutils/unstructuredutils_test.go

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


📝 Walkthrough

Walkthrough

The change removes fixed year values from SPDX copyright notices in REUSE.toml and tracked Go source, test, mock, and test-data files. License identifiers and code behavior remain unchanged.

Changes

License header migration

Layer / File(s) Summary
REUSE attribution updates
REUSE.toml
The aggregate and README.md annotations now use year-free copyright text.
Go source headers
buildutils/*, clientutils/*, cmdutils/*, conditionutils/*, configutils/*, envtestutils/*, kustomizeutils/*, metautils/*, migration/*, modutils/*, testdata/*, testutils/*, unstructuredutils/*
Production Go files now use year-free SPDX copyright notices.
Test and support headers
**/*_test.go, **/*_suite_test.go, mock/**, testdata/**
Test, mock, and support files now use year-free SPDX copyright notices.

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 0e45f

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: afritzler

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description includes the linked issue but omits the required Proposed Changes section and change summary. It is incomplete against the repository template. Add a Proposed Changes section with concise bullets describing the year-free SPDX header updates, affected files, license-marker preservation, and validation. Keep the Fixes #499 line.
Linked Issues check ❓ Inconclusive 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 … Provide evidence that all tracked files were checked, all SPDX license markers were preserved, any required rewrite process was added or documented, and make check-license passes.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the primary change: removing years from license headers.
Out of Scope Changes check ✅ Passed The changes are limited to SPDX copyright header text in tracked files and REUSE.toml. No unrelated functional or behavioral changes are identified.
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 5…
Full details: Linked Issues check

Explanation

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.

  • Fix all pre-merge checks with AI
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/499-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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove years from existing license headers

1 participant