-
Notifications
You must be signed in to change notification settings - Fork 40k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix mungegithub and the verification build #27797
Conversation
Merge first, ask questions later. |
GCE e2e build/test passed for commit e26fb81. |
@smarterclayton fyi because this fixes something about your original change. I'm not sure why the munger on your system didn't include this and also why the testing didn't start detecting it until just now. I think the latter was somehow triggered by the release @david-mcmahon just did. |
@lavalamp I'd be surprised if it was the update to the branch that caused the master munger to start failing. |
I think there's a git command that looks for release branches that maybe suddenly started working? |
Yes one of the -munge- scripts actually hardcodes looking for upstream/branch, but prior to today's push the branch already existed so that wouldn't have been 'new'. |
Exactly-- that's why I don't understand why it would have stopped working On Tue, Jun 21, 2016 at 2:48 PM, David McMahon notifications@github.com
|
No description provided.