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