Skip to content

fix(project-ideas): correct broken Physlib repo link - #19

Merged
fanza-ks merged 1 commit into
mainfrom
fix/project-ideas-repo-link
Sep 9, 2026
Merged

fix(project-ideas): correct broken Physlib repo link#19
fanza-ks merged 1 commit into
mainfrom
fix/project-ideas-repo-link

Conversation

@fanza-ks

@fanza-ks fanza-ks commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Problem

On Project Ideas, the closing sentence invites contributors to "make a pull-request to the Physlib website repo" — but the link was broken.

It was built as a template literal appending a suffix to the repo URL:

href={`${site.github}_Website`}

which resolves to https://github.com/leanprover-community/physlib_Website — a repo that does not exist, so the link 404s.

Fix

Point the anchor at site.github (https://github.com/leanprover-community/physlib), and move the link onto the words "Physlib repo" so the link text matches its destination. Previously the text read "website repo" while pointing at the library repo, which would have been misleading even once the URL was fixed.

The sentence now renders as:

…let Joseph Tooby-Smith know, or make a pull-request to the Physlib repo.

Notes

  • This was the only such concatenation in the codebase — the other site.github uses all append valid paths (/issues, /projects, /issues/new).
  • Text and a string-constant swap only; no logic change.
  • Not built or typechecked locally (node_modules not installed in this checkout) — worth letting CI/Vercel confirm.

🤖 Generated with Claude Code

The link was built as `${site.github}_Website`, concatenating a suffix
onto the repo URL and resolving to
https://github.com/leanprover-community/physlib_Website, which does not
exist.

Point it at site.github instead, and move the anchor onto "Physlib repo"
so the link text matches where it actually goes (previously the text read
"website repo" while pointing at the library repo).

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@fanza-ks
fanza-ks merged commit 55dc213 into main Sep 9, 2026
1 check passed
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