Skip to content

Conversation

@m-tmatma
Copy link
Member

@m-tmatma m-tmatma commented May 9, 2020

(必須) PR の目的

GitHub Actions の成果物を別々の ZIP として upload する

(必須) カテゴリ

  • ビルド関連
    • GitHub Actions

(自明なら省略可) PR の背景

#1259 で成果物を上げるときに、個別に上げずにまとめてアップしていた。

(自明なら省略可) PR のメリット

個別にダウンロードできる。

(なければ省略可) PR のデメリット (トレードオフとかあれば)

Installer の成果物は存在しないけど項目として存在している。

(仕様変更/機能追加の場合は必須) 仕様・動作説明

(必須) テスト内容

fork に対して push して、個別に成果物が up されること

(わかる範囲で) PR の影響範囲

GitHub Actions

(なければ省略可) 関連 issue, PR

#1259
#1263

(なければ省略可) 参考資料

@m-tmatma m-tmatma mentioned this pull request May 9, 2020
5 tasks
@m-tmatma m-tmatma force-pushed the feature/separate-artifacts-github-actions branch from 1d10132 to 70c8947 Compare May 17, 2020 08:58
@m-tmatma m-tmatma force-pushed the feature/separate-artifacts-github-actions branch from 70c8947 to 0c429b3 Compare May 25, 2020 22:41
@m-tmatma m-tmatma marked this pull request as ready for review May 25, 2020 23:29
@k-takata
Copy link
Member

actions/upload-artifact@v2 はアーティファクトを zip で固めてアップロードするので、ファイルによっては zip の中に zip が入っているのは若干無駄な感じもしますが、問題ないと思います。
ただ、アーティファクトの数がかなり多いので、MD5 を本体と一緒のアーティファクトに入れてしまうのもよいかもしれません。

Installer の成果物は存在しないけど項目として存在している。

Installer も作成するようになったので、ちゃんと含まれていますね。

@m-tmatma m-tmatma merged commit 400bf42 into sakura-editor:master May 27, 2020
@m-tmatma m-tmatma deleted the feature/separate-artifacts-github-actions branch May 27, 2020 22:09
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 16, 2020
…artifacts-github-actions

GitHub Actions の成果物を別々の ZIP として upload する
@beru beru added the CI appveyor など CI 関連 【ChangeLog除外】 label Jul 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI appveyor など CI 関連 【ChangeLog除外】

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants