Skip to content

chore: remove REUSE tooling and SAP copyright headers - #61

Merged
artemkovalyov merged 3 commits into
mainfrom
remove-reuse-v2
Sep 2, 2026
Merged

chore: remove REUSE tooling and SAP copyright headers#61
artemkovalyov merged 3 commits into
mainfrom
remove-reuse-v2

Conversation

@artemkovalyov

Copy link
Copy Markdown
Contributor

Summary

  • Removes REUSE.toml, LICENSES/Apache-2.0.txt, and the REUSE compliance GitHub Action workflow
  • Strips all SPDX-FileCopyrightText / SPDX-License-Identifier headers from source files
  • Cleans up stale references to REUSE tooling in README.md, CONTRIBUTING.md, and docs/community/index.md

Background

SPDX annotations are a standardized way to declare license and copyright per file. REUSE (by the FSFE) is a spec built on top of SPDX that requires every file in a repo to be covered — either by an inline header or a catch-all REUSE.toml. The fsfe/reuse-action CI check enforced this.

This setup made sense under SAP, which has legal requirements around per-file copyright attribution in OSS contributions. REUSE is common in the SAP OSS ecosystem for that reason.

Under the Linux Foundation umbrella (LFU), the model is different: contributors sign off commits via DCO (git commit -s), copyright stays with the original authors, and the root LICENSE file covers the whole project. Per-file SPDX headers are not required and the REUSE overhead does not add value in this governance model.

Apache 2.0 licensing remains fully in effect via the LICENSE file at the repo root — no license information is lost.

Test plan

  • Verify no REUSE workflow failures in CI (action is removed)
  • Confirm LICENSE file (Apache 2.0) is still present at repo root
  • Check README.md, CONTRIBUTING.md, and community page render correctly without stale REUSE links

CobaltCore has migrated to the Linux Foundation umbrella (LFU), making
SAP-specific copyright annotations and REUSE compliance tooling no longer
relevant. The project remains Apache 2.0 licensed via the LICENSE file.

Signed-off-by: Artem Kovalov <artem.kovalyov@gmail.com>
Copilot AI lite review requested due to automatic review settings September 2, 2026 19:16

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The updated Code of Conduct reporting guidance directs sensitive reports to public GitHub issues, which should be revised to use a private/reporting-process channel instead.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Removes REUSE/SPDX per-file compliance tooling and references from the documentation repo, aligning licensing attribution with a root-level Apache 2.0 LICENSE and dropping SAP-specific REUSE overhead.

Changes:

  • Removed REUSE configuration and the REUSE compliance GitHub Action workflow.
  • Stripped SPDX copyright/license headers from docs, workflows, and container config.
  • Cleaned up documentation references that pointed to REUSE tooling and REUSE.toml.
File summaries
File Description
REUSE.toml Removes repository-level REUSE annotations/configuration.
README.md Removes SPDX header and REUSE tool reference; updates licensing blurb.
LICENSES/Apache-2.0.txt Removes REUSE-managed license copy (root LICENSE remains).
docs/community/index.md Removes REUSE.toml maintainer-contact reference in CoC section.
Dockerfile Removes SPDX headers from container build definition.
docker-compose.yml Removes SPDX headers from compose config.
CONTRIBUTING.md Removes SPDX header and replaces REUSE.toml maintainer contact reference.
.github/workflows/update-api.yml Removes SPDX header from workflow file.
.github/workflows/static.yml Removes SPDX header from workflow file.
.github/workflows/stale.yaml Removes SPDX header from workflow file.
.github/workflows/reuse.yaml Deletes REUSE compliance check workflow.
Review details
  • Files reviewed: 11/11 changed files
  • Comments generated: 2
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread CONTRIBUTING.md Outdated
Comment thread docs/community/index.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Artem Kovalov <artem.kovalyov@gmail.com>
Copilot AI review requested due to automatic review settings September 2, 2026 19:21
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Artem Kovalov <artem.kovalyov@gmail.com>
@artemkovalyov
artemkovalyov merged commit 524500b into main Sep 2, 2026
3 checks passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The changes consistently remove REUSE/SPDX artifacts and references without leaving any remaining in-repo references or headers.

Review details
  • Files reviewed: 11/11 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

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