Uh oh!
There was an error while loading. Please reload this page.
travis: git-validation detects travis now - #366
Conversation
vbatts
commented
Apr 5, 2016
hmm. something strange happened in those failures... |
vbatts
commented
Apr 5, 2016
ah. found it. vbatts/git-validation#6 |
vbatts
commented
Apr 5, 2016
cool. PTAL. |
jonboulle
commented
Apr 5, 2016
but now why isn't this being tested?!? |
vbatts
commented
Apr 5, 2016
woah. it had been. though github was seeming to choke up a bit ago. |
vbatts
commented
Apr 5, 2016
https://travis-ci.org/opencontainers/runtime-spec/builds/120966194 ? |
vbatts
commented
Apr 5, 2016
@caniszczyk any thoughts? ^^ |
caniszczyk
commented
Apr 5, 2016
I just kicked it again, looks good? https://travis-ci.org/opencontainers/runtime-spec/builds/120966194 |
vbatts
commented
Apr 5, 2016
@caniszczyk the build is fine, but not sure why travis status is no longer showing on PRs. I'm wondering if the project rename did not sync with travis? |
caniszczyk
commented
Apr 5, 2016
@vbatts ah you're right, wth! up for creating another PR to see if the issue fixed itself with a new PR? I completely refreshed the Travis CI settings for the repos so not sure what's up yet. |
vbatts
commented
Apr 5, 2016
looks to be back. Thanks @caniszczyk |
vbatts
commented
Apr 12, 2016
PTAL |
| - go vet -x ./... | ||
| - make .golint | ||
| - $HOME/gopath/bin/git-validation -run DCO,short-subject -v -range ${TRAVIS_COMMIT_RANGE} | ||
| - git-validation -run DCO,short-subject -v |
There was a problem hiding this comment.
Maybe want dangling-whitespace too (from vbatts/git-validation#7), although that somewhat of a drive-by with this PR. Still, it would be nice to avoid things like #380.
vbatts
commented
Apr 12, 2016
this PR should fix travis failures like this https://travis-ci.org/opencontainers/runtime-spec/jobs/122597352 |
hqhq
commented
Apr 13, 2016
LGTM |
mrunalp
commented
Apr 13, 2016
Needs rebase. |
The `-range` argument is not needed. Signed-off-by: Vincent Batts <vbatts@hashbangbash.com>
mrunalp
commented
Apr 13, 2016
LGTM |
Signed-off-by: Vincent Batts vbatts@hashbangbash.com