Regenerate against the SPDX headers: every citation moves by two - #19
arpanghoshal wants to merge 2 commits into
Conversation
CTRLRun/ctrlrun#181 puts two comment lines at the top of every source file, so every line number under src/ moves by two. scripts/repoint-claims.py re-derived 151 citations in CLAIMS.md from the definitions they name against that tree; the three it cannot resolve (a float check and two DDL lines, not definitions) were moved by hand and checked. render_api.py --write regenerated the 83 API reference pages against the same tree. The table was already stale against main from today's v0.10 merges, which is why the repointer moved rows this pull request's headers did not touch. The branch carries the kernel branch's name, so CI here checks these pages against that branch and the kernel's docs job checks that branch against these pages. Signed-off-by: arpan <contact@arpanghoshal.com>
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Automations to automatically generate PRs for you. |
|
Warning Review limit reachedNext included review available in 31 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (83)
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 |
|
Correction to the description: the branch-name pairing lives in the unmerged branch |
Kernel main moved under src/ after the tree #181 was cut from, so 31 rows drifted between the first repoint and the merge. Re-derived against main as it is now; the API reference regenerated against the same tree. Signed-off-by: arpan <contact@arpanghoshal.com>
|
Superseded by #22, which regenerated CLAIMS.md and the API reference against the kernel's release/0.10.0 (now main), a newer tree than this branch's, and is merged. Nothing here is lost. |
What this changes
Pairs with CTRLRun/ctrlrun#181, which puts two comment lines at the top of every source file, so every line number under
src/moves by two.docs/CLAIMS.md:scripts/repoint-claims.pyre-derived 151 citations from the definitions they name, against the kernel branchquality/spdx-headers. The three it refuses to guess at (afloatcheck inaction.pyand a DDL line inmigrations.py, which are not definitions) were moved by hand and checked against the same tree.docs/reference/api/*.mdx:render_api.py --writeregenerated the 83 pages against the same tree;--checkreports 0 drifted.The table was already stale against
mainfrom today's v0.10 merges (97 rows), which is why the repointer also moved rows the headers did not touch; that drift would have surfaced on the nextlibrary-changeddispatch.This branch carries the kernel branch's name, so CI here checks these pages against that kernel branch, and the kernel's
docsjob on #181 checks that branch against these pages. Merge order: #181 there first, then this; between the two,mainhere is red on the claims test for a few minutes.tests/test_claims_and_adapters.pyandtests/test_claims_citations.pypass against the kernel branch.🤖 Generated with Claude Code