Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

manuals: Fix position of SPDX license line #1466

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gperciva
Copy link
Contributor

No description provided.

Signed-off-by:	Graham Percival <[email protected]>
Sponsored by:	Tarsnap Backup Inc.
Pull Request:	freebsd#1466
@gperciva
Copy link
Contributor Author

A very low-priority PR.

Based on @bsdimp's comment [1], I added a lint check to my doc scripts [2] to check the SPDX line. Since there were only 9 non-contrib files that were incorrect, I manually edited them.

[1] #1458 (comment)
[2] https://github.com/Tarsnap/freebsd-doc-scripts/blob/main/lint.py#L14

@bsdimp
Copy link
Member

bsdimp commented Oct 13, 2024

These are all likely good, but their sources are (a) jhb and (b) the foundation, so we may want to chat with john and likely ed or someone he redirects us to... and it would likely be good for me to create a canonical source of this guidance.

@gperciva
Copy link
Contributor Author

Sure thing! I viewed this as starting / continuing a conversation, not the end point.

FWIW, the only problems in contrib/ are the 32 files in contrib/libfido2/man/*.[1-9]. Everything else passes the linter with no complaints.

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