chore: add owner SPDX header to all .rs (satisfy licence-enforcer hook)#139
Merged
Conversation
…l test rename The pre-commit licence enforcer requires BOTH SPDX-License-Identifier: MPL-2.0 AND the owner string 'Jonathan D.A. Jewell <j.d.a.jewell@open.ac.uk>' on every staged source file, but NO .rs file carried the owner line -- so any edit to a .rs file was blocked by the hook (a latent repo-wide gap). Inserts the SPDX-FileCopyrightText owner line on line 2 of all 91 tracked .rs files (each already had SPDX-License-Identifier on line 1). Also renames the cosmetic 'audience.md' temp filename in the axial test to 'axial.md', completing the audience->axial sweep. Comment-only changes plus one test-string rename; cargo check --all-targets passes. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
hyperpolymath
added a commit
that referenced
this pull request
Jun 24, 2026
…r/.zig) (#141) Completes the licence-enforcer-hook compliance started for `.rs` in #139. The pre-commit hook requires **both** `SPDX-License-Identifier: MPL-2.0` **and** the owner string `Jonathan D.A. Jewell <j.d.a.jewell@open.ac.uk>` on every staged `.md`/`.adoc`/`.idr`/`.zig`/`.rs` source. After #139, the `.rs` tree was clean but 45 non-`.rs` sources still lacked the hook-matching owner line (15 also lacked the SPDX line) — so any edit to those would be hook-blocked. ## Changes - Inserts the `SPDX-FileCopyrightText` owner line (and `SPDX-License-Identifier` where absent) into **47 files** (35 `.md`, 10 `.adoc`, 6 `.idr`, 2 `.zig` — minus the 8 already compliant), using each file's comment syntax: HTML comment for `.md`, `//` for `.adoc`/`.zig`, `--` for `.idr`. - **Fully non-destructive: 64 insertions, 0 deletions.** No existing attribution line is altered. Files that already carry a `(hyperpolymath)`-parenthetical copyright keep it; the hook-matching line is added alongside (REUSE permits multiple `SPDX-FileCopyrightText`). A follow-up could normalise the two forms if desired. All **145** hook-checked files in the repo now carry SPDX + owner. Comment-only changes — no rendered/compiled output affected. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The pre-commit licence enforcer (
.git/hooks/pre-commit) requires bothSPDX-License-Identifier: MPL-2.0and the exact owner stringJonathan D.A. Jewell <j.d.a.jewell@open.ac.uk>on every staged source file — but no.rsfile carried the owner line. Every.rshad the SPDX identifier on line 1, yet the owner check (stricter than the existing tree) meant any future edit to a.rsfile was blocked by the hook. Surfaced while fixing the audience→axial doc drift (#137).Changes
// SPDX-FileCopyrightText: 2026 Jonathan D.A. Jewell <j.d.a.jewell@open.ac.uk>on line 2 of all 91 tracked.rsfiles (each already had the SPDX identifier on line 1).audience.mdtemp filename in theaxialtest toaxial.md, completing the audience→axial sweep.Comment-only changes plus one test-string rename.
cargo check --all-targetspasses; the licence-enforcer hook now passes on.rsedits (this commit itself passed it with hooks enabled).Out of scope (noted, not done):
.md/.adoc/.idrsource files — the two header-less docs are handled in #137 and the new Idris2 fixture in #138 already carries the header; a follow-up can extend the same owner line to remaining non-.rssources if wanted.🤖 Generated with Claude Code