Skip to content

seisho v2 に上げ、PDF を pdf/ に置く - #8

Merged
5ym merged 1 commit into
masterfrom
seisho-v2
Oct 3, 2026
Merged

5ym merged 1 commit into
masterfrom
seisho-v2

Conversation

@5ym

@5ym 5ym commented Oct 3, 2026

Copy link
Copy Markdown
Owner

seisho v2 では、PDF を pdf ブランチではなく同じブランチの pdf/ に置き、Action がコミットする。

マージすると最初の実行で 29 本が pdf/academic/... に作られる。古い pdf ブランチはマージ後に消す。

🤖 Generated with Claude Code

https://claude.ai/code/session_01USCeESvgLneCai7g8399GM

pdf ブランチは使わなくなるので消す。

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01USCeESvgLneCai7g8399GM
@claude

claude Bot commented Oct 3, 2026 •

Copy link
Copy Markdown

Claude finished @5ym's task in 14s —— View job


レビュー結果

変更は seisho@v1 → @v2 への更新と README の記述修正のみで、ブロッカーとなる問題はありません。

確認した点

  • convert.yml は permissions: contents: write を持っているため、Action が同じブランチへ pdf/ をコミットできます。
  • トリガーは push の master で、GITHUB_TOKEN による push は workflow を再起動しません。出力先の pdf/ は入力の academic/ とも別なので、無限ループの心配はありません。
  • concurrency: pdf により、連続 push 時の競合も避けられます。
  • README のリンク pdf/ は、マージ後の最初の実行で作られるまでリンク切れになります。ただし PR 本文で説明済みなので問題としません。

要確認(コードからは判断できない点)

  • 5ym/seisho@v2 のタグが存在し、pdf/ へのコミットに対応していること。
  • master にブランチ保護 (PR 必須など) がある場合、GITHUB_TOKEN による push が拒否される可能性があります。マージ後の最初の実行が成功したかを確認してください。
  • 古い pdf ブランチの削除は、29 本が pdf/academic/... に作られたことを確認してからにするのが安全です。

テストは対象外です (設定とドキュメントのみの変更)。

@5ym
5ym merged commit 5d73195 into master Oct 3, 2026
1 check passed
@5ym
5ym deleted the seisho-v2 branch October 3, 2026 12:30
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