Expand pgferry.com docs surface for issue #69#70
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Code Review SummaryPositives
Issues to Address1. CI - Missing dependency between # .github/workflows/ci.yml:214
needs: [test, integration-mysql, integration-sqlite, integration-mysql-postgis, integration-mssql]The 2. Verification script - Hardcoded expected routes 3. .gitignore change is incomplete 4. Missing 5. Documentation - No mention of Phase 2 6. Minor: verify-build.mjs line 112 if (pathname.endsWith('/')) {Should be Recommendations
Overall: Good documentation additions, but the verification script has a bug that will cause CI failures. |

Closes #69
This draft PR makes
pgferry.comthe primary end-user docs surface instead of a thin shell that points users back to GitHub for the real answer.What changed
reference/*coverage so the missing reference routes build againREADME.mdinto a concise project entry point that points users to the website for full docsdocs/*.mdend-user docs in favor of a small pointer pagevercel.json,robots.txt, and stronger canonical/sitemap verificationactions/setup-nodebefore Bun so Astro gets a supported runtime on GitHub ActionsVerification
bun install --frozen-lockfilebun run buildbun run check-routesCommits are intentionally split into phases:
9709559Add site reference docs and route checksc9ddc44Expand site docs IA and example coverage84031aaRetire repo docs and harden site deploy configfab6879Set Node 22 for site CI builds38d46bdUse Node lts in site CI jobb65f40bAdd operator guidance follow-up pagesRemaining external follow-up
www.pgferry.comis attached in Vercel, configure the domain redirect there so the live host behavior matches the apex canonical URLs emitted by the site.