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

dev版リリースをHomebrewでインストールできるようにする #9

Merged
merged 3 commits into from
Nov 10, 2023
Merged

dev版リリースをHomebrewでインストールできるようにする #9

merged 3 commits into from
Nov 10, 2023

Conversation

umi1299
Copy link
Contributor

@umi1299 umi1299 commented Nov 10, 2023

内容

VOICEVOXのdev版リリース (nightlyリリース?) をHomebrewでインストールできるようにします

@umi1299
Copy link
Contributor Author

umi1299 commented Nov 10, 2023

失敗しているCIがありますが無視して大丈夫なエラーなので気にしないでください🙇‍♂️

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

ちなみにどうしてエラーが出た感じなんでしょう・・・? 👀

@@ -0,0 +1,30 @@
cask "voicevox-dev" do
version "0.15.0-dev"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

正式版の0.15.0が出たタイミングで、この値は0.16.0-devに変更する必要があるかもです!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

0.16.0-devがリリースされたらPRが自動で作成されるはずなので大丈夫と思います

@Hiroshiba Hiroshiba merged commit 8191064 into VOICEVOX:main Nov 10, 2023
3 of 6 checks passed
@umi1299
Copy link
Contributor Author

umi1299 commented Nov 10, 2023

@Hiroshiba マージありがとうございます!

ちなみにどうしてエラーが出た感じなんでしょう・・・? 👀

CIに brew audit --cask というチェック処理があるのですが、これはpre-releaseのリリースを指定するとエラーを出すようになっています
なので brew audit --cask を省略した ci-preview.yml とそれを省略していない ci.yml を用意しています
この2つは pathspaths-ignore によってどちらを走らせるかを分岐させているのですが、今回は両方に関わる差分が入っているので全てのcaskに対して brew audit --cask が走ってしまってエラーになっていました
自動PRのときはこのようなことは起こらないので無視して問題ないと思います
ただ、すみませんが最小バージョンを指定していないエラーも出ていて、これはまずいかもしれないのでまた出たら対処を考えようと思います!

@Hiroshiba
Copy link
Member

なるほどです!!
ちゃんと理解できてないですが、リポジトリのREADME.mdを変更した場合なども発動してテストがエラーになる・・・?

テストが落ちるのに慣れると、テストが異常終了していることに気づかなくなりがちという問題が実はあったりします。

といっても変更は簡単かも・・・?
例えばCI.ymlの方を

    paths-ignore:
      - Casks/*-dev.rb
      - Casks/*-preview.rb

から

    paths:
      - Casks/voicevox.rb

に変えると解決しそうでしょうか 👀

@umi1299
Copy link
Contributor Author

umi1299 commented Nov 11, 2023

テストでは変更されたcaskしか見ないのでREADME.mdの変更でエラーになることはないですね
ただ、Casks/voicevox.rb の以外の変更のみで ci.yml が走る必要はなかったので提案いただいた (↓) ようにしようと思います!

例えばCI.ymlの方を

    paths-ignore:
      - Casks/*-dev.rb
      - Casks/*-preview.rb

から

    paths:
      - Casks/voicevox.rb

に変えると解決しそうでしょうか 👀

@Hiroshiba
Copy link
Member

おっとなるほどです!!
Github Actionsの負荷をあげないように心がけるためにも切れるなら切っておきたいですね・・・!

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

Successfully merging this pull request may close these issues.

2 participants