Every source file carries its copyright and licence in SPDX form - #181
Conversation
Two comment lines at the top of every .py and .sh under src/, tests/, fuzz/, scripts/, adapters/ and examples/: the copyright holder (the contributors) and the licence (Apache-2.0), in the SPDX form a tool can read. A shebang stays on line one. LICENSE at the root is unchanged and still the licence text. A file copied out of this tree on its own now says what it is. The test in test_repository_signals.py walks the same six directories and fails on any source file without the two lines, so the next file added carries them too. The claims table in CTRLRun/ctrlrun-docs cites line numbers under src/, and every one moves by two; the matching pull request there shifts them and regenerates the API reference pages against this commit. Signed-off-by: arpan <contact@arpanghoshal.com>
|
Important Review skippedToo many files! This PR contains 166 files, which is 66 over the limit of 100. To get a review, reduce the PR to 100 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (166)
You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
What this changes
Two comment lines at the top of every
.pyand.shundersrc/,tests/,fuzz/,scripts/,adapters/andexamples/:SPDX-FileCopyrightText: 2026 The CTRLRun contributorsandSPDX-License-Identifier: Apache-2.0. A shebang stays on line one. No code changes;ruff format --checkandruff checkare clean on the tree, and the examples suite passes.Why: a file copied out of this tree on its own should say who holds it and under what licence, in the form tools read. This is the
copyright_per_fileandlicense_per_filepair of the OpenSSF Best Practices gold criteria.Line numbers under
src/move by two.CLAIMS.mdinCTRLRun/ctrlrun-docscites them, so thedocsjob here is red until the matching pull request there merges (it shifts every citation and regenerates the API reference pages against this commit). Merge this first, then that one.Checklist
test_every_source_file_carries_its_copyright_and_licensewalks the six directories and fails on any file without the two lines.CLAIMS.md. No README sentence changes; the citations shift in the paired docs pull request.scripts/check.shgreen for the parts a comment can touch:ruff format --check,ruff check, and the repository and examples suites.src/merges on green CI alone; a maintainer reads it.Mutation table
src/ctrlrun/errors.pytest_every_source_file_carries_its_copyright_and_license🤖 Generated with Claude Code