Skip to content

quartz: fix baseUrl typo (graitools -> gridtools) - #37

Merged
egparedes merged 1 commit into
mainfrom
fix-pages-baseurl
Oct 1, 2026
Merged

egparedes merged 1 commit into
mainfrom
fix-pages-baseurl

Conversation

@egparedes

Copy link
Copy Markdown
Contributor

quartz.config.ts had:

baseUrl: "graitools.github.io/gt4py_knowledge",

but the live Pages URL for this repo is https://gridtools.github.io/gt4py_knowledge/:

$ gh api repos/GridTools/gt4py_knowledge/pages --jq '{html_url, cname}'
{"cname":null,"html_url":"https://gridtools.github.io/gt4py_knowledge/"}

graitools is a transposition of gridtools. AGENTS.md states that baseUrl must
match the final GitHub Pages URL of the repo, and Quartz uses it for canonical URLs,
the sitemap, RSS and open-graph metadata — so absolute links the published site emits
currently point at a host that does not resolve.

One line, no content change. Found while investigating something unrelated in
quartz.config.ts; kept separate from #36 deliberately.

`baseUrl` read `graitools.github.io/gt4py_knowledge`, but the GitHub Pages URL
reported by the API for this repo is `https://gridtools.github.io/gt4py_knowledge/`.
AGENTS.md requires the two to match, and Quartz uses `baseUrl` for canonical URLs,
the sitemap, RSS and open-graph links, so every absolute link the published site
emits currently points at a host that does not exist.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The single configuration typo is corrected with no unresolved issues.

Review effort: Lite
Findings: None

What changed in this PR

Fixes the Quartz baseUrl hostname typo so generated URLs target the live GitHub Pages site.

Changes:

  • Replaces graitools.github.io with gridtools.github.io.
File Description
quartz.config.ts Corrects the GitHub Pages base URL.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@egparedes
egparedes merged commit c5b0bdc into main Oct 1, 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.

2 participants