Skip to content

fix: footer "contribute on GitHub" link points to CONTRIBUTING.md#41

Merged
guillaumegay13 merged 1 commit into
mainfrom
chore/audit-external-links
Jun 1, 2026
Merged

fix: footer "contribute on GitHub" link points to CONTRIBUTING.md#41
guillaumegay13 merged 1 commit into
mainfrom
chore/audit-external-links

Conversation

@brunobuddy
Copy link
Copy Markdown
Member

Summary

  • The footer link labelled "contribute on GitHub" used to send visitors to tree/main/models — the raw catalog data — when the label promises contribution guidance.
  • Switched the target to blob/main/CONTRIBUTING.md so people who click the contribute link actually land on the guide that explains how to add models, write parameters, and open a PR.
  • Audited the other external GitHub links across header.ejs, how_to_use.ejs, the README, src/data/llms.ts, and src/build/build.ts. The mnfst/modelparams.dev and anomalyco/models.dev references all resolve and match their intended labels, so no other URL changes were needed.

Test plan

  • npm run typecheck
  • npm test (67/67 passing)
  • npm run lint
  • npm run validate (140 models)
  • Spot-check the footer link renders correctly in the built page and opens the CONTRIBUTING guide

The link labelled "contribute on GitHub" in the footer used to drop
visitors into the models/ directory listing, which is the data, not the
contribution guide. Send them to CONTRIBUTING.md instead so the label
matches the destination.
@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 1, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
modelparams.dev Ready Ready Preview Jun 1, 2026 7:03am

Request Review

@github-actions github-actions Bot added the site Website code or tooling label Jun 1, 2026
@guillaumegay13 guillaumegay13 merged commit fd3dc05 into main Jun 1, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

site Website code or tooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants