kkimurak / ssl-rules-ja

Unofficial Japanese translation of RoboCup Small Size League Rule Book (https://github.com/robocup-ssl/ssl-rules)
https://kkimurak.github.io/ssl-rules-ja/
GNU General Public License v3.0
3 stars 0 forks source link

Revert "ci: install htmldiffer from github@develop branch, specify file encoding" #88

Closed kkimurak closed 8 months ago

kkimurak commented 8 months ago

本PRは以下の変更を差し戻します: https://github.com/kkimurak/ssl-rules-ja/commit/aa88b94e472c3dc88610179e660a81f4773c9532

84 でCircleCIへの移行に追従しましたが、日本語版リポジトリとして独自の変更 aa88b94e472c3dc88610179e660a81f4773c9532 を加えていました。これは私の手元の環境(日本語Windowsマシン)で動かしていたときに遭遇した文字化けへの対処で、入出力のエンコードをUTF-8と明示してやることで解決を試みるもので、手元の環境では問題が解消していました。

また差分生成のために使っているPythonライブラリのhtmldifferで該当する機能(ファイルのエンコーディング指定)が最新のリリース(といっても2017年)に含まれていなかったため、pip installでgithubからインストールするようにしていました。

しかしCircleCIに移行したところCIが落ちており(たとえば https://app.circleci.com/jobs/github/kkimurak/ssl-rules-ja/12)、この処理が原因であったのではないかと推測されました。また処理時間が異様に長く、本家ルールのCIではpip installを含めても3秒で完了しているにもかかわらず本リポジトリでは4分かかっています。手元のWindowsで走らせたときもこの程度の差は認められました。

kkimurak commented 8 months ago

master-jpにマージしないと動作がわからないため、さっさとマージして様子を見ます。

kkimurak commented 8 months ago

駄目でした。前回同様、gh-pagesブランチをcheckoutして今回のビルド成果物html,pdfで上書きしているはずなのにnothing to commitになります。

https://app.circleci.com/pipelines/github/kkimurak/ssl-rules-ja/15/workflows/ded924aa-c085-460b-a991-bd7d4e69e324/jobs/15