Skip to content

chore: add SPDX license headers and NOTICE file#29

Merged
amrtgaber merged 1 commit into
mainfrom
chore/license-headers
Jun 15, 2026
Merged

chore: add SPDX license headers and NOTICE file#29
amrtgaber merged 1 commit into
mainfrom
chore/license-headers

Conversation

@amrtgaber

Copy link
Copy Markdown
Contributor

What

  • Add SPDX license headers (Apache-2.0, AG Technology Group LLC) to every tracked Python source file (35 files).
  • Add an attribution-only NOTICE at the repo root.

Enforcement (CI-only)

  • scripts/license_header.py — dependency-free, stdlib-only check/stamp tool.
    • The CI lint job runs python scripts/license_header.py --check.
    • Run python scripts/license_header.py --fix to stamp new files locally.
  • Deliberately not wired into .pre-commit-config.yaml. This is a template repo: a local hook would be inherited by every clone and have to be ripped out. The check only verifies that an SPDX-License-Identifier line exists, so a downstream fork with its own copyright still passes.

Skipped

Generated-banner files, vendored dirs, and non-source files (configs, lockfiles, start.sh).

Verification

ruff check / ruff format --check clean, pytest (32) passing, and the check is idempotent (35 stamped / 0 missing).

Stamp every Python source file with SPDX headers (Apache-2.0,
AG Technology Group LLC) and add an attribution-only NOTICE.

Enforcement is CI-only (no local hook): the lint job runs
`python scripts/license_header.py --check`, which verifies each tracked
.py file carries an SPDX identifier (run --fix to stamp new files).
Kept out of pre-commit so anyone cloning the template doesn't inherit a
local hook they'd have to strip out.
@amrtgaber amrtgaber merged commit 03fd490 into main Jun 15, 2026
2 checks passed
@amrtgaber amrtgaber deleted the chore/license-headers branch June 15, 2026 02:45
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.

1 participant