ビルドを全通り試さない簡易テストモードを追加する#554
Merged
Hiroshiba merged 11 commits intoVOICEVOX:mainfrom Jul 31, 2023
Hidden character warning
The head ref may contain hidden characters: "\u30d3\u30eb\u30c9\u3092\u5168\u901a\u308a\u8a66\u3055\u306a\u3044\u7c21\u6613\u30c6\u30b9\u30c8\u30e2\u30fc\u30c9\u3092\u8ffd\u52a0\u3059\u308b"
Merged
Conversation
Hiroshiba
commented
Jul 29, 2023
| run: msbuild /m /p:Configuration=${{env.BUILD_CONFIGURATION}} ${{env.SOLUTION_FILE_PATH}} | ||
|
|
||
| build-python-api: | ||
| build-and-test-python-api: |
Comment on lines
-268
to
-269
| env: | ||
| CARGO_TERM_COLOR: always |
Hiroshiba
commented
Jul 29, 2023
| target_commitish: ${{ github.sha }} | ||
|
|
||
| build_xcframework: | ||
| if: ${{ !(github.event_name != 'release' && github.event_name != 'workflow_dispatch') }} # !env.IS_SIMPLE_TEST と同じ |
Member
Author
There was a problem hiding this comment.
ここではenvを使えないのでこういう風に書くしかない･･･。
qryxip
reviewed
Jul 30, 2023
Member
Author
|
レビューありがとうございます！ 問題ないと思うのでマージします！！ これでちょっと開発者と環境に優しくなったはず。 |
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
2 participants
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.
内容
の一環です。
プッシュするたびにかなり多くの環境と条件でビルドタスクが回っていて、その影響でテストがすごく時間かかっていました。
このプルリクエストでは簡易テストという概念を導入し、各OSにつき1つだけビルドテストするようにしてみました。
workflow_dispatch（Github Actionsから直接実行できるやつ）とreleaseの経路では全部のビルドを行います。
それ以外の経路（push・pull_request）では簡易テストとします。
関連 Issue
その他
条件matrixを動的に変更する必要があるのですが、現状のgithub actionsでは前段階としてジョブを実行し、その出力結果を持ってmatrixを変更する以外方法がありません。
なので前段階のジョブ側に全条件のjsonを作っておいてフィルタリングするようにしました。
さすがに見づらかったのでジョブごとに改行を導入しました。