Skip to content

refactor: move script and extract build-docs action#12

Merged
TaiSakuma merged 5 commits intomainfrom
refactor/move-script
Mar 21, 2026
Merged

refactor: move script and extract build-docs action#12
TaiSakuma merged 5 commits intomainfrom
refactor/move-script

Conversation

@TaiSakuma
Copy link
Owner

Summary

  • Move override_site_url.py from scripts/ to .github/scripts/
  • Extract common build step into .github/actions/build-docs/ composite action
  • Modify zensical.toml in place instead of creating a copy
  • Output deploy URL from deploy-to-gh-pages action, eliminating the separate "Get preview URL" step

Test plan

  • PR preview deploys correctly with comment
  • Push to main deploys /dev/

🤖 Generated with Claude Code

TaiSakuma and others added 4 commits March 21, 2026 14:56
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The deploy action now outputs the default GitHub Pages URL,
eliminating the separate "Get preview URL" step.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions github-actions bot added the refactor Code refactoring label Mar 21, 2026
The in-place modification of zensical.toml blocks the subsequent
git checkout to gh-pages. Restore it after building since the
built site is already copied to /tmp.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Contributor

github-actions bot added a commit that referenced this pull request Mar 21, 2026
@TaiSakuma TaiSakuma merged commit 0fafff2 into main Mar 21, 2026
2 checks passed
@TaiSakuma TaiSakuma deleted the refactor/move-script branch March 21, 2026 19:19
github-actions bot added a commit that referenced this pull request Mar 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Code refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant