GitHub Actionsでビルドする#60
Merged
Merged
Conversation
Owner
|
おおっ!ありがとうございます!確認します🖐🖐🖐 |
miiton
approved these changes
May 18, 2020
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Cica を GitHub Actionsでビルドする実装をしてみました。並列ビルド (#53) するにしても、まずは今のビルド処理をそのまま動かせるようにするのが第一歩でしょうし、まずはそこまで PR で貢献できたら良いなと思っています。もしよろしければ accept してください。
2点、補足です。
Hack-Regular.ttfなど) の置き場を指すCICA_SOURCE_FONTS_PATHという環境変数を導入し、これを使うようにcica.pyを変更しています/Dockerfileを実行することでビルドしています。docker run --workdir /github/workspace ...を実行)、Dockerfile のWORKDIRが結果的に無視されてしまいます。その結果、./sourceFontsという相対パスを使っているcica.pyはソースフォントを見つけられずエラーになります。cica.pyの修正内容が現在進行中の PR カスタマイズ用のCLIオプションを追加 #52 とコンフリクトします。なので カスタマイズ用のCLIオプションを追加 #52 を先に完了すると、この PR に修正が必要になりますが、それはこちらでやりますのでご連絡ください。よろしくお願いします。
追伸:GitHub Actions 実行例はこちらでご確認ください。