Merge pull request #1134 from net2b/skip-ci

Allow any case/seq to be used for “[CI SKIP]”
This commit is contained in:
Brad Rydzewski 2015-08-12 08:03:51 -07:00
commit 57b009e3c4

View File

@ -3,6 +3,7 @@ package handler
import ( import (
"log" "log"
"net/http" "net/http"
"regexp"
"strings" "strings"
"github.com/drone/drone/plugin/remote" "github.com/drone/drone/plugin/remote"
@ -43,7 +44,8 @@ func PostHook(c web.C, w http.ResponseWriter, r *http.Request) {
// in some cases we have neither a hook nor error. An example // in some cases we have neither a hook nor error. An example
// would be GitHub sending a ping request to the URL, in which // would be GitHub sending a ping request to the URL, in which
// case we'll just exit quiely with an 'OK' // case we'll just exit quiely with an 'OK'
if hook == nil || strings.Contains(hook.Message, "[CI SKIP]") { shouldSkip, _ := regexp.MatchString(`\[(?i:ci *skip|skip *ci)\]`, hook.Message)
if hook == nil || shouldSkip {
w.WriteHeader(http.StatusOK) w.WriteHeader(http.StatusOK)
return return
} }