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

#1185 での rebase の誤りを修正 #1187

Merged

Conversation

m-tmatma
Copy link
Member

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

PR の目的

#1185 での rebase の誤りを修正

カテゴリ

  • 不具合修正
  • CI関連
    • Appveyor
    • Azure Pipelines

PR の背景

#1183 で PR を分割して #1185 を作成する際に、rebase 操作で変更を分割する際に
appeyor 側の修正ではなく、あやまって azure pipeline の修正を入れてしまった。

その部分のみ修正する。

PR のメリット

#1185 がマージされた状態で GitHub コミット URL の機能が動作しなくなる問題が修正される。

PR のデメリット (トレードオフとかあれば)

なし

PR の影響範囲

appveyor ビルドでの GitHub コミット URL

関連チケット

#1185
#1183

参考資料

@m-tmatma
Copy link
Member Author

m-tmatma commented Feb 9, 2020

#1183 から PR を切り出すときの誤りで、 #1183 自体には影響のないものです。

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.

見逃してました。さーせん 😢

一点気になるのは、BUILD を含む環境変数は azure pipelines由来な気がしていて、既に入ってる行を削らなくていいんだっけ?ということです。

一連のPRで最終的なつじつまが合えば、細かいとこはいいような気もしますが。

@m-tmatma
Copy link
Member Author

m-tmatma commented Feb 9, 2020

一点気になるのは、BUILD を含む環境変数は azure pipelines由来な気がしていて、既に入ってる行を削らなくていいんだっけ?ということです。

いいことにします。

@m-tmatma
Copy link
Member Author

m-tmatma commented Feb 9, 2020

見逃してました。さーせん 😢

私が確認を端折ったのが原因です。

@m-tmatma
Copy link
Member Author

m-tmatma commented Feb 9, 2020

https://ci.appveyor.com/project/sakuraeditor/sakura/builds/30672806/job/9crfte5e7865dx9c/artifacts

の sakura-PR1187-build2612-2c50ae8-Win32-Release-Log.zip で githash.h 確認して OK でした。

sakura-PR1187-build2612-2c50ae8-Win32-Release-Exe.zip でも Github へのリンクに飛べるのを確認しました。

@m-tmatma
Copy link
Member Author

m-tmatma commented Feb 9, 2020

CI 終わったらマージします。

@AppVeyorBot
Copy link

Build sakura 1.0.2612 completed (commit 2c50ae8882 by @m-tmatma)

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.

4 participants