Skip to content

refactor: define SITE_DIR once in PR preview workflow#19

Merged
TaiSakuma merged 2 commits intomainfrom
dev
Mar 22, 2026
Merged

refactor: define SITE_DIR once in PR preview workflow#19
TaiSakuma merged 2 commits intomainfrom
dev

Conversation

@TaiSakuma
Copy link
Owner

Summary

  • Define SITE_DIR as a workflow-level env var in docs-pr-preview.yml
  • Add site-dir input to build-docs action (with default /tmp/built-site)
  • Use ${{ env.SITE_DIR }} consistently in all four places (build, upload, download, deploy)

Test plan

  • PR preview deploys correctly after merge

🤖 Generated with Claude Code

TaiSakuma and others added 2 commits March 22, 2026 06:09
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Extract /tmp/built-site into a workflow-level env var and pass it
to build-docs, upload/download-artifact, and deploy-to-gh-pages.
Add site-dir input to build-docs action.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions github-actions bot added the refactor Code refactoring label Mar 22, 2026
github-actions bot added a commit that referenced this pull request Mar 22, 2026
@github-actions
Copy link
Contributor

@TaiSakuma TaiSakuma merged commit 2e8fd98 into main Mar 22, 2026
4 checks passed
@TaiSakuma TaiSakuma deleted the dev branch March 22, 2026 10:26
github-actions bot added a commit that referenced this pull request Mar 22, 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