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

artifacts の名前に appveyor の名前を含める (構成要素の順序調整) #234

Conversation

m-tmatma
Copy link
Member

@m-tmatma m-tmatma commented Jul 8, 2018

artifacts の名前に appveyor の名前を含める (構成要素の順序調整)

以下の表の順番で構成する。

構成要素 意味 元の変数名 必須
"sakara" prefix なし
BUILD_ACCOUNT appveyor アカウント名 APPVEYOR_ACCOUNT_NAME × "sakuraeditor" の場合 またはローカルビルドの場合、なし
TAG_NAME tag 名 APPVEYOR_REPO_TAG_NAME ×
PR_NAME PR 番号 APPVEYOR_PULL_REQUEST_NUMBER ×
BUILD_NUMBER build Number APPVEYOR_BUILD_NUMBER ○ (ローカルビルドの場合 "buildLocal")
SHORTHASH hash 値の先頭8文字 APPVEYOR_REPO_COMMIT ×
platform CPU platform platform
configuration configuration configuration
RELEASE_PHASE alpha バージョンかどうか ALPHA × (x64 のみ)

影響範囲

  • artifacts の zip ファイル名
  • artifacts の zip ファイルの内部フォルダの名前

@m-tmatma m-tmatma added the CI appveyor など CI 関連 【ChangeLog除外】 label Jul 8, 2018
@m-tmatma m-tmatma added this to the next release milestone Jul 8, 2018
@m-tmatma
Copy link
Member Author

m-tmatma commented Jul 8, 2018

最終的なコミットでは appveyor.yml の先頭に "*" を付ける必要はありませんが、
修正の途中で必要になって修正しました。削除したほうが良ければ削除します。

appveyor.yml Outdated
@@ -21,4 +21,4 @@ build_script:
echo appveyor_yml

artifacts:
- path: 'sakura-$(platform)-$(configuration)*.zip'
- path: '*sakura-*$(platform)-$(configuration)*.zip'
Copy link
Contributor

Choose a reason for hiding this comment

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

コメントで言及してるのはこれのことですか?
zipの出力先をルートにしたから*は要らない、と。
分かっている内容なら削除したほうがいいような気もしますが、直すまでもないと思います。

Copy link
Member Author

Choose a reason for hiding this comment

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

後で必要になるかも、というのがあったので残しておきました。
d2c8255 で削除しました。

Copy link
Contributor

@berryzplus berryzplus left a comment

Choose a reason for hiding this comment

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

LGTMです。

アカウント名追加の確認はPR元で行いました。

@m-tmatma m-tmatma merged commit 5da6469 into sakura-editor:master Jul 8, 2018
@m-tmatma m-tmatma deleted the feature/add-appveyor-username-to-artifacts branch July 8, 2018 09:07
@ds14050 ds14050 added the CI appveyor など CI 関連 【ChangeLog除外】 label Sep 18, 2018
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
…or-username-to-artifacts

artifacts の名前に appveyor の名前を含める (構成要素の順序調整)
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