Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Jul 16, 2015

  • Easier testing for outsiders....

@ghost ghost mentioned this pull request Jul 16, 2015
@alex-fedorov
Copy link
Collaborator

@dg-ratiodata I figured it out, why ruby_22 is not available: #180

@alex-fedorov
Copy link
Collaborator

@dg-ratiodata try to rebase, should work now

@ghost
Copy link
Author

ghost commented Jul 16, 2015

Re-push to trigger rebuild

@ghost
Copy link
Author

ghost commented Jul 16, 2015

Finished. All green.

@alex-fedorov
Copy link
Collaborator

I have a feeling, that re-pushing is not enough. You need to rebase against upstream master. This might help: https://gist.github.com/waterlink/ee7600e499d2ca8c596f

@alex-fedorov
Copy link
Collaborator

Hmm, I don't quite understand how you did it green..

@ghost
Copy link
Author

ghost commented Jul 16, 2015

Magic... No, just git commit --amend && git push -f Then travis sees a new commit and starts again.

@alex-fedorov
Copy link
Collaborator

I see. Our travis setup uses pr/XXX/merge instead of pr/XXX/head. And I think it should use both.

@alex-fedorov
Copy link
Collaborator

Ok, I will rebase manually and merge it. I don't want to make merge diversions in the tree.

@ghost
Copy link
Author

ghost commented Jul 16, 2015

ok. Thanks.

@alex-fedorov
Copy link
Collaborator

Merged manually

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.

1 participant