Skip to content

docs: fix stale explainer.html README comment - #340

Merged
yakew7 merged 2 commits into
yakew7:mainfrom
lovishmenaria14-gif:fix-explainer-readme-comment
Aug 29, 2026
Merged

docs: fix stale explainer.html README comment#340
yakew7 merged 2 commits into
yakew7:mainfrom
lovishmenaria14-gif:fix-explainer-readme-comment

Conversation

@lovishmenaria14-gif

@lovishmenaria14-gif lovishmenaria14-gif commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Fix the stale explainer.html repository-tree comment in README.md.

The previous comment incorrectly described explainer.html as an explainer-page template rendered by scripts/build_explainers.py. In reality, it is a static ?slug= redirect shim that points to explainers/<slug>.html.

This updates the README comment to accurately reflect the current implementation and references DEAD-FILE-AUDIT.md for additional context.

Closes #326

Testing

Documentation-only change. No code changes or tests are required.

@vercel

vercel Bot commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

@lovishmenaria14-gif is attempting to deploy a commit to the yashkewlani2020-gmailcom's projects Team on Vercel.

A member of the Team first needs to authorize it.

@yakew7

yakew7 commented Aug 29, 2026

Copy link
Copy Markdown
Owner

Could you js edit the pr body to add what issue it closes

@yakew7
yakew7 merged commit c33e58c into yakew7:main Aug 29, 2026
16 of 17 checks passed
yakew7 added a commit that referenced this pull request Aug 29, 2026
…) to CODEOWNERS

PR #340 (a README.md fix) triggered no review-request ping - none of
these files had a CODEOWNERS entry, so GitHub had nothing to route to.
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.

README.md's repo-tree comment for explainer.html is stale — it's not "rendered per slug by build_explainers.py"

2 participants