Skip to content

Add automatic translation Actions file.#35

Open
KaiTomotake wants to merge 2 commits into
rust-developers-jp:mainfrom
KaiTomotake:add-automatic-translation
Open

Add automatic translation Actions file.#35
KaiTomotake wants to merge 2 commits into
rust-developers-jp:mainfrom
KaiTomotake:add-automatic-translation

Conversation

@KaiTomotake
Copy link
Copy Markdown
Member

概要

自動翻訳システムを追加

変更内容

  • .github/workflows/translation.ymlを追加

レビューで見てほしい点

初めてまともなActionsを書いたので、本当にこれで動くのかどうかをよく見てほしい。

@KaiTomotake KaiTomotake requested a review from hinshiba May 27, 2026 22:26
@Aqua-218
Copy link
Copy Markdown
Member

Aqua-218 commented May 28, 2026

初回から schedule 有効なのはちょっと怖いです。
まずは workflow_dispatch だけで入れて手動実行で確認したいです。

あと、MDX 本文と翻訳結果を $GITHUB_OUTPUT 経由で渡しているのも少し気になります。出力結果が大きくなると output のサイズ制限で落ちそうなので、ここはファイル経由にしたほうがよさそうです。

細かいですが、cat ${{ steps.pick.outputs.target }}> $path あたりは quote しておいたほうが安全そうです。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants