Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Push transifex ワークフローの作成 #17

Open
sanak opened this issue Jan 12, 2021 · 7 comments
Open

Push transifex ワークフローの作成 #17

sanak opened this issue Jan 12, 2021 · 7 comments
Assignees

Comments

@sanak
Copy link
Member

sanak commented Jan 12, 2021

update-omegat_ja のPRをマージした際のイベントをトリガーにし、PR変更差分のファイル名を読み取って、以下のWikiに記載のPush操作を自動で対応するワークフローを作成します。
https://github.com/OSGeo-jp/OSGeoLive-doc/wiki#push-transifex

(QGIS翻訳の方で指摘のあった、Transifexとの共存案(1ファイル毎にPR作成、Transifex push時に差分をチェック)については、別のIssue/PRで2月以降に対応の予定です。)

@sanak sanak self-assigned this Jan 12, 2021
@sanak
Copy link
Member Author

sanak commented Jan 26, 2021

@miurahr (CC: @smellman)
社内で、OSGeoLive-doc の全体翻訳メンテナンスをしている Vicky(@cvvergara) から、今回の日本語翻訳の進捗が驚愕に値するとのことでした。
OSGeo@2d995eb

Transifexの更新が私の名前だけになっていまずが、やはり実際に翻訳した人の名前が上がる形が良いようなので、下記に加えて、マージ実行者のTransifexアカウントでPushできる形が望ましいかもしれません。

(QGIS翻訳の方で指摘のあった、Transifexとの共存案(1ファイル毎にPR作成、Transifex push時に差分をチェック)については、別のIssue/PRで2月以降に対応の予定です。)

(なお、当初、GItHub の Secrets に各自、TransifexのAPIトークンを TX_TOKEN_(アカウント名) を切り替えることを考えてましたが、 ${{ secrets.TX_TOKEN_(アカウント名) }} を指定できるかどうかが怪しくて、 pull-transifex.yml の方では断念しました...。)

@sanak
Copy link
Member Author

sanak commented Feb 6, 2021

1ファイル毎のPR作成は、GitHub Actionsのワークフロー内でループ処理を組めなさそうなので、断念します...。
(ひょっとすると、専用のアクションを自作(https://docs.github.com/ja/actions/creating-actions) すれば可能かもしれませんが、私の方は手が回りそうにないです...。)

Secretsの方も、 ${{ secrets.TX_TOKEN_(アカウント名) }} の指定が難しい場合は断念する形になるかもしれません...。

@sanak
Copy link
Member Author

sanak commented Feb 6, 2021

Secretsの ${{ secrets.TX_TOKEN_(アカウント名) }} は問題なさそうでしたので、下記でマージします。

sanak added a commit that referenced this issue Feb 6, 2021
#17 Supported dynamic TX_TOKEN secrets by github.actor
@sanak
Copy link
Member Author

sanak commented Feb 6, 2021

冒頭(当初目的)の下記については、Transifex push中に失敗した際の判別が付かなくなるので、マージ後でなく、マージ前に自動か手動かでワークフローを実行する形とします。

update-omegat_ja のPRをマージした際のイベントをトリガーにし、PR変更差分のファイル名を読み取って、以下のWikiに記載のPush操作を自動で対応するワークフローを作成します。
https://github.com/OSGeo-jp/OSGeoLive-doc/wiki#push-transifex

@miurahr
Copy link
Collaborator

miurahr commented Feb 6, 2021

これは、本リポジトリのSecretsに、 TX_TOKEN_MIURAHR を作成し、TransifexのAPI TOKENを登録するとよいのでしょうか。

@sanak
Copy link
Member Author

sanak commented Feb 6, 2021

@miurahr

これは、本リポジトリのSecretsに、 TX_TOKEN_MIURAHR を作成し、TransifexのAPI TOKENを登録するとよいのでしょうか。

はい。
ただ、念のため、 TX_TOKEN_miurahr と、GitHubアカウント名箇所を大文字・小文字の区別ありで登録頂いた方が良いかもしれません。
(私も TX_TOKEN_sanak で登録しましたが、GitHubの設定画面では、何故か TX_TOKEN_SANAK と表示されてしまってます...。)

設定後、下記の Pull Transifex アクションから、Run workflow をクリックし、緑の Run workflow ボタンをクリックして、実際の動作を確認できると思います。

@miurahr
Copy link
Collaborator

miurahr commented Feb 11, 2021

Secret作成しました。

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

No branches or pull requests

2 participants