-
Notifications
You must be signed in to change notification settings - Fork 313
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
build.ymlのbuild部分とupload部分を一貫させる #1257
The head ref may contain hidden characters: "upload-distributable-to-release\u3092\u89E3\u4F53"
build.ymlのbuild部分とupload部分を一貫させる #1257
Conversation
.github/workflows/build.yml
Outdated
@@ -440,21 +470,53 @@ jobs: | |||
run: | | |||
df -h | |||
|
|||
- name: Upload Linux AppImage artifact | |||
- name: Create Linux AppImage splitted |
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.
もともとはbuild.yml側でAppImageを作ってupload側でsplitしていたのを、どちらもここで行うように変更しました
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.
こういう便利Action、別リポジトリで持っておいてVOICEVOX Org内の他からも気軽に使えるようにしたいかも。
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!
typoと表記が気になったのでコメントしました!
最低限typoを直してマージで良いと思います...!
すみません、メンテナ判断的なところに返事できていませんでした... |
Co-authored-by: Yuto Ashida <[email protected]>
with: | ||
files: ". .github" |
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.
@y-chan typosが.githubに対して働いていなかったので追加してみました。
ほんとは_typos.toml
側にこの設定を追加しておくと再利用性が上がって便利そうですが、なかったのでこっちに。。
問題ないと思うのでマージします! typosは一般化して全リポジトリから良い感じに使えるように切り出したいですね・・・ |
runs: | ||
using: "composite" | ||
steps: | ||
- name: Upload to Artifacts | ||
if: inputs.artifact_name != '' | ||
uses: actions/upload-artifact@v3 | ||
with: | ||
name: ${{ inputs.artifact_name }} | ||
path: ${{ inputs.files }} | ||
|
||
- name: Upload to Release Assets | ||
uses: softprops/action-gh-release@v1 | ||
with: | ||
prerelease: ${{ inputs.prerelease }} | ||
tag_name: ${{ inputs.tag_name }} | ||
files: ${{ inputs.files }} | ||
target_commitish: ${{ inputs.target_commitish }} |
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.
upload_release
とupload_artifact
のフラグが使われてない!
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.
修正PR出しました #1261
内容
を一旦完了させるタスクです。
build.yml内に書かれていた多段のjobが、これで1つのjobになります。
artifactにアップロード・ダウンロードする工程がなくなり、遅いjobを待つこともないため全体的に早くなりました。
以前のAction(54min)
https://github.com/Hiroshiba/voicevox/actions/runs/4454000489
PRのActoin(26min)
https://github.com/Hiroshiba/voicevox/actions/runs/4455163321
関連 Issue
close #837
その他
実際にRelease assetsを作ってみたのの比較です。
以前の https://github.com/Hiroshiba/voicevox/releases/tag/0.15.0-noreleasejob
PRの https://github.com/Hiroshiba/voicevox/releases/tag/0.15.0-noreleasejob.2
実際にどちらもダウンロードしてアーカイブファイルを展開し、ファイル名が一致することを確認しました。
確認コード
@y-chan メンテナ共有です。

linux-nvidiaのtar.gzがほぼほぼ2GBになっていて、次のアップデートのときに容量オーバーになりそうだったので、ついでにアップロードしないようにしてみました。
もともとたまに容量オーバーしていたのでVOICEVOXページからのリンクは省いていたのもあって、あまり問題ないかなと思ってますがどうでしょう 👀