Skip to content

🍕 Fix docs deploy URL parsing in GitHub Actions#108

Merged
jjpaulino merged 2 commits into
masterfrom
jordan/docs-deploy-url-fix
Apr 23, 2026
Merged

🍕 Fix docs deploy URL parsing in GitHub Actions#108
jjpaulino merged 2 commits into
masterfrom
jordan/docs-deploy-url-fix

Conversation

@jjpaulino
Copy link
Copy Markdown
Member

Summary

  • remove the custom clay/docusaurus-github-action deploy flow that builds an invalid repository URL in CI
  • run docs versioning and build with explicit npm commands in website/
  • publish docs to gh-pages using peaceiris/actions-gh-pages with GITHUB_TOKEN

Why

The previous deploy step failed with an invalid URL error (Invalid port in url) during docs publish. This switches to a standard GitHub Pages deploy path that avoids that URL construction bug.

Test plan

  • Trigger Deploy to GitHub Pages workflow on this branch
  • Confirm docs build succeeds and gh-pages publish step completes

Made with Cursor

Replace the custom Docusaurus deploy action with explicit build steps and a standard gh-pages publish action to avoid invalid repo URL parsing in CI.

Made-with: Cursor
@jjpaulino jjpaulino merged commit 35bfa9d into master Apr 23, 2026
4 checks passed
@jjpaulino jjpaulino deleted the jordan/docs-deploy-url-fix branch April 23, 2026 14:20
jjpaulino added a commit that referenced this pull request Apr 29, 2026
The docusaurus build/deploy in push.yml has been broken since the URL
parsing fix in #108 / install fix in #109 — the docs site is no
longer being maintained as a separate publishing target, and the
workflow was firing on every master push without a real consumer.

Removing it (and its required gh-pages permissions) trims one CI
job per merge and avoids the false-failure noise. The docs source
under website/ stays in the repo and can be rebuilt locally if/when
a docs site is brought back online.

Made-with: Cursor
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