1
0
Fork 0
mirror of https://github.com/vbatts/git-validation.git synced 2024-09-26 12:24:06 +00:00

main: Explicit -range implies -no-travis

If someone went through the trouble to set -range, they probably don't
want you clobbering their value with something from a TRAVIS_*
environment variable.
This commit is contained in:
W. Trevor King 2017-01-26 21:13:02 -08:00
parent 206708c3bd
commit 0e135a4759

14
main.go
View file

@ -14,7 +14,7 @@ import (
)
var (
flCommitRange = flag.String("range", "", "use this commit range instead")
flCommitRange = flag.String("range", "", "use this commit range instead (implies -no-travis)")
flListRules = flag.Bool("list-rules", false, "list the rules registered")
flRun = flag.String("run", "", "comma delimited list of rules to run. Defaults to all.")
flVerbose = flag.Bool("v", false, "verbose")
@ -48,11 +48,13 @@ func main() {
}
var commitRange = *flCommitRange
if strings.ToLower(os.Getenv("TRAVIS")) == "true" && !*flNoTravis {
if os.Getenv("TRAVIS_COMMIT_RANGE") != "" {
commitRange = os.Getenv("TRAVIS_COMMIT_RANGE")
} else if os.Getenv("TRAVIS_COMMIT") != "" {
commitRange = os.Getenv("TRAVIS_COMMIT")
if commitRange == "" {
if strings.ToLower(os.Getenv("TRAVIS")) == "true" && !*flNoTravis {
if os.Getenv("TRAVIS_COMMIT_RANGE") != "" {
commitRange = os.Getenv("TRAVIS_COMMIT_RANGE")
} else if os.Getenv("TRAVIS_COMMIT") != "" {
commitRange = os.Getenv("TRAVIS_COMMIT")
}
}
}