Suppot deploy status - Githubissues

wantedly / step-pretty-slack-notify

Posts wercker build/deploy status to a Slack channel

MIT License

88 stars 34 forks source link

Closed lazyppp closed 10 years ago

lazyppp commented 10 years ago

こんにちは。 step-pretty-slack-notify 毎日便利に使っています。すばらしい plugin のご提供ありがとうございます。

werckerで、deploy の機能も使っているので、 deploy の場合にもstatus メッセージを出せるようにしました。簡単な仕様は下記の通りです。

・buildの場合従来どおり・deployの場合

[lazyppp/sample] deploy(e79f4788) of master to production by lazyppp passed

もしよろしければ、mergeしていただけると嬉しいです。

spesnova commented 10 years ago

PR ありがとうございます :exclamation: デプロイの方も対応しないとなぁと思っていたので助かります :)

lazyppp commented 10 years ago

コメントありがとうございます。おっしゃる通り修正しました。

ところで、今更なんですが、コミットメッセージミスっていることに気づきました。 (誤) Suppot deploy status (正) Support deploy status

いちから修正して、もう一度違うbranchでpullrequestしても問題ないでしょうか？

spesnova commented 10 years ago

いちから修正して、もう一度違うbranchでpullrequestしても問題ないでしょうか？

問題ないですよ〜。ではこの PR は閉じちゃいますね :octocat: