Skip to content

Docs: drop the License section from template READMEs - #46

Closed
deividasma wants to merge 1 commit into
mainfrom
chore/readme-no-license-section
Closed

deividasma wants to merge 1 commit into
mainfrom
chore/readme-no-license-section

Conversation

@deividasma

Copy link
Copy Markdown
Contributor

What

  • 31 template READMEs: removed the ## License section. It was always the same two lines, "MIT. See LICENSE; reuse and adapt this template…", right before ## Screenshots. Nothing else changed: exactly 5 deleted lines per file.
  • create-repository skill:
    • RULES.md: the license lives in the LICENSE file and the package.json license field, not in the README.
    • RULES.md and SKILL.md: the README must not add sections that README_TEMPLATE.md doesn't have, such as License.

Why

README_TEMPLATE.md never had a License section, but agents kept adding one: 31 of 51 templates, plus Dex's #45. The skill says to follow the template exactly, so these READMEs were already off-spec. The LICENSE file and the package.json field are unchanged, so every template still states its reuse terms.

Notes

🤖 Generated with Claude Code

README_TEMPLATE.md never had a License section, yet 31 templates added
one. The LICENSE file and package.json license field already state the
terms, so remove the README copies and say so in the create-repository
skill.
@deividasma deividasma closed this Oct 19, 2026
@github-actions

Copy link
Copy Markdown

Template preview

Built each changed template, opened npm run preview, requested /, then compared Playwright screenshots to the committed catalog images (preview/<id>-thumbnail.png at 1440×900 and preview/<id>-homepage.png full page).

Template Install Lint Build Audit Health Thumbnail diff Homepage diff
api-reference ✅ ✅ ✅ ✅ ✅ 2.3% ✅ 1.6% ✅
bakery-preorder ✅ ✅ ✅ ✅ ✅ 1.0% ✅ 1.0% ✅
callout ✅ ✅ ✅ ✅ ✅ 1.2% ✅ 1.1% ✅
careers ✅ ✅ ✅ ✅ ✅ 0.8% ✅ 1.1% ✅
changelog ✅ ✅ ✅ ✅ ✅ 5.8% ✅ 9.7% ✅
class-timetable ✅ ✅ ✅ ✅ ✅ 2.0% ✅ 1.9% ✅
client-gallery ✅ ✅ ✅ ✅ ✅ 0.9% ✅ 0.6% ✅
console ✅ ✅ ✅ ✅ ✅ 1.5% ✅ 2.2% ✅
couch-to-5k ✅ ✅ ✅ ✅ ✅ 1.0% ✅ 1.7% ✅
design-system ✅ ✅ ✅ ✅ ✅ 1.5% ✅ 1.5% ✅
devfolio ✅ ✅ ✅ ✅ ✅ 1.9% ✅ 2.2% ✅
drop ✅ ✅ ✅ ✅ ✅ 9.4% ✅ 9.4% ✅
engineering-blog ✅ ✅ ✅ ✅ ✅ 2.0% ✅ 2.5% ✅
first-chapter ✅ ✅ ✅ ✅ ✅ 0.9% ✅ 1.4% ✅
flash-sheet ✅ ✅ ✅ ✅ ✅ 0.4% ✅ 1.2% ✅
groom-room ✅ ✅ ✅ ✅ ✅ 0.9% ✅ 0.6% ✅
manual ✅ ✅ ✅ ✅ ✅ 2.4% ✅ 3.4% ✅
movement-library ✅ ✅ ✅ ✅ ✅ 1.3% ✅ 1.4% ✅
moving-day ✅ ✅ ✅ ✅ ✅ 0.4% ✅ 7.4% ✅
podcast-home ✅ ✅ ✅ ✅ ✅ 2.2% ✅ 3.9% ✅
quickstart ✅ ✅ ✅ ✅ ✅ 2.0% ✅ 3.2% ✅
quiet-practice ✅ ✅ ✅ ✅ 2 pre-existing ✅ 1.0% ✅ 1.6% ✅
quote-slider ✅ ✅ ✅ ✅ ✅ 22.2% ✅ 18.6% ✅
roadmap ✅ ✅ ✅ ✅ ✅ 3.2% ✅ 2.3% ✅
route-library ✅ ✅ ✅ ✅ ✅ 1.2% ✅ 1.5% ✅
sunday-league ✅ ✅ ✅ ✅ ✅ 17.3% ✅ 11.1% ✅
supper-club ✅ ✅ ✅ ✅ ✅ 0.6% ✅ 0.7% ✅
tap-list ✅ ✅ ✅ ✅ ✅ 1.8% ✅ 1.7% ✅
todays-menu ✅ ✅ ✅ ✅ ✅ 2.8% ✅ 7.8% ✅
trust-centre ✅ ✅ ✅ ✅ ✅ 1.8% ✅ 5.8% ✅
will-it-survive ✅ ✅ ✅ ✅ ✅ 1.6% ✅ 1.2% ✅

api-reference

Expected thumbnail Actual Diff
expected actual diff

bakery-preorder

Expected thumbnail Actual Diff
expected actual diff

callout

Expected thumbnail Actual Diff
expected actual diff

careers

Expected thumbnail Actual Diff
expected actual diff

changelog

Expected thumbnail Actual Diff
expected actual diff

class-timetable

Expected thumbnail Actual Diff
expected actual diff

client-gallery

Expected thumbnail Actual Diff
expected actual diff

console

Expected thumbnail Actual Diff
expected actual diff

couch-to-5k

Expected thumbnail Actual Diff
expected actual diff

design-system

Expected thumbnail Actual Diff
expected actual diff

devfolio

Expected thumbnail Actual Diff
expected actual diff

drop

Expected thumbnail Actual Diff
expected actual diff

engineering-blog

Expected thumbnail Actual Diff
expected actual diff

first-chapter

Expected thumbnail Actual Diff
expected actual diff

flash-sheet

Expected thumbnail Actual Diff
expected actual diff

groom-room

Expected thumbnail Actual Diff
expected actual diff

manual

Expected thumbnail Actual Diff
expected actual diff

movement-library

Expected thumbnail Actual Diff
expected actual diff

moving-day

Expected thumbnail Actual Diff
expected actual diff

podcast-home

Expected thumbnail Actual Diff
expected actual diff

quickstart

Expected thumbnail Actual Diff
expected actual diff

quiet-practice

Expected thumbnail Actual Diff
expected actual diff

quote-slider

Expected thumbnail Actual Diff
expected actual diff

roadmap

Expected thumbnail Actual Diff
expected actual diff

route-library

Expected thumbnail Actual Diff
expected actual diff

sunday-league

Expected thumbnail Actual Diff
expected actual diff

supper-club

Expected thumbnail Actual Diff
expected actual diff

tap-list

Expected thumbnail Actual Diff
expected actual diff

todays-menu

Expected thumbnail Actual Diff
expected actual diff

trust-centre

Expected thumbnail Actual Diff
expected actual diff

will-it-survive

Expected thumbnail Actual Diff
expected actual diff

Download screenshots and diffs from this run’s visual-<template> artifacts.

This check fails if install, lint, build, / health, or screenshot diff over the limit fails, or if the PR introduces a high/critical advisory in production dependencies that the base commit did not have (npm audit --omit=dev, head vs base). Pre-existing advisories are listed for information and belong to dependency-update PRs. Limits default to 25% (thumbnail) and 30% (homepage) to absorb Mac-vs-Linux font rasterization.

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