Merge pull request #418 from abdullin/master
Tweaking slack integration to include commit message
This commit is contained in:
commit
3e3367cc97
1 changed files with 10 additions and 4 deletions
|
@ -8,9 +8,9 @@ import (
|
||||||
|
|
||||||
const (
|
const (
|
||||||
slackEndpoint = "https://%s.slack.com/services/hooks/incoming-webhook?token=%s"
|
slackEndpoint = "https://%s.slack.com/services/hooks/incoming-webhook?token=%s"
|
||||||
slackStartedMessage = "*Building* %s, commit <%s|%s>, author %s"
|
slackStartedMessage = "*Building* %s <%s|%s>, by %s:\n> %s"
|
||||||
slackSuccessMessage = "*Success* %s, commit <%s|%s>, author %s"
|
slackSuccessMessage = "*Success* %s <%s|%s>, by %s:\n> %s"
|
||||||
slackFailureMessage = "*Failed* %s, commit <%s|%s>, author %s"
|
slackFailureMessage = "*Failed* %s <%s|%s>, by %s:\n> %s"
|
||||||
)
|
)
|
||||||
|
|
||||||
type Slack struct {
|
type Slack struct {
|
||||||
|
@ -47,7 +47,13 @@ func getBuildUrl(context *Context) string {
|
||||||
|
|
||||||
func getMessage(context *Context, message string) string {
|
func getMessage(context *Context, message string) string {
|
||||||
url := getBuildUrl(context)
|
url := getBuildUrl(context)
|
||||||
return fmt.Sprintf(message, context.Repo.Name, url, context.Commit.HashShort(), context.Commit.Author)
|
return fmt.Sprintf(
|
||||||
|
message,
|
||||||
|
context.Repo.Name,
|
||||||
|
url,
|
||||||
|
context.Commit.HashShort(),
|
||||||
|
context.Commit.Author,
|
||||||
|
context.Commit.Message)
|
||||||
}
|
}
|
||||||
|
|
||||||
func (s *Slack) sendStarted(context *Context) error {
|
func (s *Slack) sendStarted(context *Context) error {
|
||||||
|
|
Loading…
Reference in a new issue