docs: add third-party attribution NOTICE for upstream CV template - #20
Merged
Merged
Conversation
…demic CV LaTeX Template
… discrepancy details
eduardstan
added a commit
that referenced
this pull request
Jul 31, 2026
* docs: add third-party attribution notice for Dubasi Pavan Kumar's Academic CV LaTeX Template * docs: refine third-party notice with exact Overleaf URL and licensing discrepancy details * no-mistakes(review): drop root CLAUDE.md symlink, point LICENSE at NOTICE * no-mistakes(document): docs: point preamble editors at upstream attribution NOTICE
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
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.
Intent
Credit upstream academic CV LaTeX template by Dubasi Pavan Kumar with third-party notice
What Changed
NOTICEcrediting Dubasi Pavan Kumar's Academic CV Template as the origin ofcv/preamble.texand the LaTeX layouts undercv/, with the source URL and the MIT-vs-CC BY 4.0 discrepancy between the upstream file header and the Overleaf listing, plus how this repository satisfies both.LICENSEatNOTICEwith a one-line reference instead of duplicating the attribution text, and updates README's "Licence" section and "Make it yours" entry to linkNOTICEand warn that the upstream header incv/preamble.texmust survive edits.cv/preamble.texmarking the existing upstream header as load-bearing attribution.Review flagged a stray root
CLAUDE.md -> AGENTS.mdsymlink (double-loading the same instructions, against AGENTS.md's rule) and the duplicated licence block; both were auto-fixed on the branch. Docs-only — no build, check or output behaviour changes.Risk Assessment
✅ Low: Docs-only attribution change: NOTICE holds the third-party credit once, LICENSE keeps a one-line pointer, README links both, the upstream header in cv/preamble.tex is intact, and the previously flagged root CLAUDE.md symlink is gone.
Testing
Ran the one existing test that touches repository doc paths (scripts/agents-md.test.mjs, passing), then verified the attribution the way a reader meets it: rendered the README "Licence" section, the new NOTICE and the LICENSE tail in a browser and captured a screenshot, confirmed the relative NOTICE link resolves, checked that the NOTICE's claim about cv/preamble.tex retaining the upstream MIT header is true, and fetched and screenshotted the live Overleaf listing, which states exactly the author and CC BY 4.0 licence the NOTICE cites. No test was added because AGENTS.md forbids builds failing over credit or branding. Everything checked passes and the worktree is unchanged.
/tmp/no-mistakes-evidence/01KYTCWCTHBFVK63SAY3B5TSPH/attribution-surfaces.png)/tmp/no-mistakes-evidence/01KYTCWCTHBFVK63SAY3B5TSPH/upstream-overleaf-listing.png)Evidence: Verification transcript
$ git ls-files NOTICE LICENSE README.md CLAUDE.md LICENSE NOTICE README.md $ grep -n 'NOTICE' README.md LICENSE LICENSE:24:Third-party attributions: see NOTICE. README.md:529:NOTICE. Bundled web fonts retain their own licences $ sed -n '1,9p' cv/preamble.tex % %Academic CV LaTeX Template % Author: Dubasi Pavan Kumar % LinkedIn: https://www.linkedin.com/in/im-pavankumar/ % License: MIT $ curl -sSL https://www.overleaf.com/latex/templates/academic-cv-template/gmyytjmdbvdm http=200 Dubasi Pavan Kumar Creative Commons CC BY 4.0Evidence: Local render page used for the screenshot
Pipeline
Updates from git push no-mistakes
✅ **intent** - passed
✅ No issues found.
✅ **Rebase** - passed
✅ No issues found.
🔧 **Review** - 2 issues found → auto-fixed ✅
CLAUDE.md:1- Commit 6fdf78c adds a rootCLAUDE.md -> AGENTS.mdsymlink, which directly contradicts the committed rule in AGENTS.md:143-145: "Keep.claude/CLAUDE.mdas the single one-line import of this file for Claude-compatible tools. Do not add another pointer at the repository root: having both pointers makes those tools load the same instructions twice.".claude/CLAUDE.mdstill exists and imports../AGENTS.md, so Claude-compatible tools now load the same instructions twice. It is also unrelated to the stated intent (third-party attribution) and looks like a stray file swept into the docs commit. Either delete the symlink or update AGENTS.md's rule deliberately.LICENSE:22- The Third-Party Notices block appended to LICENSE (lines 22-32) is byte-identical to the block in the new NOTICE file. Two copies of the same legal text drift apart on any future edit, and appending non-MIT prose to LICENSE lowers licensee/GitHub license-detection similarity, so the repo can stop being reported as MIT. Keep the block only in NOTICE and reduce the LICENSE addition to a one-line pointer ("Third-party attributions: see NOTICE").🔧 Fix: drop root CLAUDE.md symlink, point LICENSE at NOTICE
✅ Re-checked - no issues remain.
✅ **Test** - passed
✅ No issues found.
node --test scripts/agents-md.test.mjs— the only test covering repository doc paths (passes; root CLAUDE.md symlink removal breaks nothing,.claude/CLAUDE.mdpointer intact)git ls-files NOTICE LICENSE README.md CLAUDE.md— NOTICE is tracked so it ships in the tracked-file-only adopter copy; root CLAUDE.md is gonegrep -n 'NOTICE' README.md LICENSE— both surfaces reference the new filesed -n '1,9p' cv/preamble.tex— verifies the NOTICE's own claim that the upstream% Author: Dubasi Pavan Kumar/% License: MITheader is retainedManual render: served the repo's real NOTICE and LICENSE plus the README "Licence" section over http://127.0.0.1:8731 and screenshotted it in Chrome; the relativeNOTICElink resolves (HTTP 200)curl -sSL https://www.overleaf.com/latex/templates/academic-cv-template/gmyytjmdbvdmplus a Chrome screenshot — upstream listing is live and shows Author: Dubasi Pavan Kumar, License: Creative Commons CC BY 4.0, corroborating the NOTICE text✅ **Document** - passed
✅ No issues found.
✅ **Lint** - passed
✅ No issues found.
✅ **Push** - passed
✅ No issues found.