Skip to content

ci: add REUSE compliance lint as PR status check#103

Merged
ricogu merged 5 commits into
mainfrom
feat/reuse
May 26, 2026
Merged

ci: add REUSE compliance lint as PR status check#103
ricogu merged 5 commits into
mainfrom
feat/reuse

Conversation

@mdanish98

@mdanish98 mdanish98 commented May 26, 2026

Copy link
Copy Markdown
Member

Summary

Adds a GitHub Actions workflow that runs reuse lint on every PR targeting main, enforcing REUSE specification compliance for copyright and license information across all files. Also fixes existing coverage gaps in REUSE.toml so the initial check passes from day one.

Closes #92

Changes

Added

  • .github/workflows/reuse-lint.yml — new REUSE Compliance workflow using fsfe/reuse-action@v5, triggered on pull_request to main

Technical Details

The project already had REUSE.toml with a LICENSES/ directory in place. The gap was that .opencode/** (opencode tooling + its node_modules) and openspec/** (change management artifacts) were not annotated, causing reuse lint to report ~3400 uncovered files.

The fix uses precedence = "override" for the node_modules glob so it takes priority over the broader .opencode/** annotation, allowing "their respective authors" attribution for third-party code without changing the project-owned files in .opencode/.

reuse lint was verified locally before this PR — all 3474 files pass (3474/3474 compliant).

Compatibility & Impact

Breaking Changes

  • None

Dependencies

  • Relies on fsfe/reuse-action@v5 (external GitHub Action, FSFE-maintained)

Testing

  • reuse lint run locally and confirmed clean (3474/3474 files, no errors)
  • Confirm reuse-lint status check appears and passes on this PR

Comment thread .github/workflows/reuse-lint.yml Fixed
Comment thread .github/workflows/reuse-lint.yml Fixed
@mdanish98 mdanish98 requested a review from ricogu May 26, 2026 08:55

@ricogu ricogu 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.

please see my comments

Comment thread LICENSES/MPL-2.0.txt Outdated
Comment thread REUSE.toml Outdated
ricogu
ricogu previously approved these changes May 26, 2026
@ricogu ricogu enabled auto-merge (squash) May 26, 2026 09:00
@ricogu ricogu disabled auto-merge May 26, 2026 09:01
@mdanish98 mdanish98 closed this May 26, 2026
@mdanish98 mdanish98 reopened this May 26, 2026
@ricogu ricogu enabled auto-merge (squash) May 26, 2026 09:10
@ricogu ricogu disabled auto-merge May 26, 2026 09:27
@ricogu ricogu merged commit 865b4a1 into main May 26, 2026
13 checks passed
@ricogu ricogu deleted the feat/reuse branch May 26, 2026 09:27
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.

ci: Add REUSE lint as PR status check

3 participants