chore: add NOTICE carving brand fonts out of the MIT grant - #43
Merged
Conversation
Prep for making this repo public. The root LICENSE grants MIT over everything, including the Base Sans brand typefaces under public/fonts/ — which would let anyone reuse Base's brand typeface as their own. Base owns the fonts, so this is a brand decision rather than a licensing defect, but the blanket grant should not extend to them. NOTICE reserves all rights on public/fonts/BaseSans* and points forkers at removing them (including the next/font/local declarations, since deleting the files alone fails the build). It also records the SIL OFL terms for Doto and, separately, the MIT attribution for viem/ox/abitype/@noble that vendor/aa/index.js inlines — the bundler strips upstream notices, which MIT requires be retained. LICENSE and README now point at NOTICE. Co-Authored-By: Claude <noreply@anthropic.com>
Collaborator
🟡 Heimdall Review Status
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
2 participants
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.
Prep for making this repo public.
Why
The root
LICENSEgrants MIT over the whole repo — including the Base Sans brandtypefaces in
public/fonts/. MIT grantsuse, copy, modify, merge, publish, distribute, sublicense, and/or sell, so as written it lets anyone take Base's brandtypeface and use it as their own. Base owns the fonts, so this is a brand decision
rather than a licensing defect, but the blanket grant shouldn't extend to them.
Precedent supports carving them out: the public
base-org/brand-kitdeliberatelyomits fonts, and other Base repos shipping Base Sans are private.
What
NOTICE(new) — reserves all rights onpublic/fonts/BaseSans*andBaseSansMono*, explicitly excluding them from the MIT grant. Tells forkers how toremove them (deleting the files alone fails the build, since they're loaded via
next/font/localinapp/layout.tsx; the CSS stack inglobals.cssalready hassystem fallbacks).
public/fonts/Doto/(already compliant via itsnametable; documenting it for clarity).
vendor/aa/index.js, which inlinesviem / ox / abitype / @noble under MIT. The bundler strips upstream copyright
notices, and MIT requires they be retained in substantial portions — this restores
that attribution.
LICENSEandREADMEnow point atNOTICE.No code changes.
npm run buildandnpm run docs:checkpass.