Skip to content

Conversation

@vbatts
Copy link
Member

@vbatts vbatts commented Mar 21, 2017

cc @mrunalp
I'm also going to try this as the upstream default vbatts/git-validation#15

Signed-off-by: Vincent Batts [email protected]

@vbatts
Copy link
Member Author

vbatts commented Mar 21, 2017

oh boo. the new range includes merge commits ... hrm

@vbatts
Copy link
Member Author

vbatts commented Mar 21, 2017

@mrunalp restart the travis check. I think i fixed it upstream

@mrunalp
Copy link
Contributor

mrunalp commented Mar 21, 2017

@vbatts restarted build.

@vbatts
Copy link
Member Author

vbatts commented Mar 21, 2017

@mrunalp it's green now

@mrunalp
Copy link
Contributor

mrunalp commented Mar 21, 2017

LGTM

Approved with PullApprove

@crosbymichael
Copy link
Member

crosbymichael commented Mar 21, 2017

LGTM

Approved with PullApprove

@crosbymichael crosbymichael merged commit a4c49f5 into opencontainers:master Mar 21, 2017
@cyphar
Copy link
Member

cyphar commented Mar 21, 2017

It looks like this broke the Travis CI builds...

@cyphar cyphar mentioned this pull request Mar 21, 2017
46 tasks
@vbatts
Copy link
Member Author

vbatts commented Mar 21, 2017 via email

@wking
Copy link
Contributor

wking commented Mar 21, 2017 via email

@vbatts vbatts deleted the fix_travis_var branch March 21, 2017 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants