-
Notifications
You must be signed in to change notification settings - Fork 309
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
zipファイルをreleaseにアップロードする #656
The head ref may contain hidden characters: "zip\u30D5\u30A1\u30A4\u30EB\u3092release\u306B\u30A2\u30C3\u30D7\u30ED\u30FC\u30C9\u3059\u308B"
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#656 (comment) のCIが落ちています。
upload-distributable-to-release
のmatrix.artifact_name
の項目が不足しています。
新しく追加した*-zip
を追加する必要があります。
おそらく冗長なのでmatrix.os
やmatrix.artifact_name
を消してしまってもいいかもです(includeだけにする)。
@aoirint 修正しました!! とりあえずコード量が最小になりそうな方法にしてみました。 それと #638 (comment) にもあるように、linuxはtargzのが良さそうなのでそうするように修正したいと思います! |
linuxはtargzを作るようにし、actionsを回し始めてみました 2022/01/23 18:41 やりなおしました |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
targzの対応もありがとうございます!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
内容
今までzip版は、artifactからダウンロードしてからgoogle driveにアップロードしていました。
結構煩雑で時間がかかっていたので、zipファイルが2GB未満になったのもあって、releasesにアップロードすることにしたいです。
関連 Issue
close #636
close #638
その他
zipファイルの容量は大体1.8GBで、github releasesの制限の2GBギリギリです。
electronのアップデートなどでいつかこの制限を越してしまう可能性はあります。
超してしまったら悲しみつつgoogle driveにアップロードする運用に戻すことにして、2GB未満の間はreleasesにアップロードとしたいと思います。