Skip to content
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

Merge with upstream #2

Closed
wants to merge 546 commits into from
Closed

Merge with upstream #2

wants to merge 546 commits into from

Conversation

mernst
Copy link

@mernst mernst commented Jul 7, 2016

The exception-handling changes in Daikon are based on an 11-month-old version of Daikon. It has to be merged with upstream before pull request codespecs#56 can be merged. Furthermore, that pull request needs to be up to date to permit meaningful code review.

Unfortunately, the exception-handling changes are 546 commits behind master, making merging them problematic. I have spent a day trying to get the merging right, and I think this attempt is close. However, it does not pass its tests. Can you review and improve this pull request, or do the merging yourself so that the exception-handling pull request codespecs#56 is based on the current version of Daikon?

Thanks!

mernst and others added 30 commits March 3, 2016 21:38
@rarworld
Copy link
Owner

rarworld commented Jul 8, 2016

Hi, this is the wrong branch.
The pull request codespecs#56 is from the FeatureExceptionHandling-Branch.
This is just the old working-Branch. But the other branch was also a little bit old. I just updated it

@rarworld rarworld closed this Jul 8, 2016
@mernst
Copy link
Author

mernst commented Jul 9, 2016

That's confusing! Thanks for the clarification. I noticed the pull request was out of date but wasn't careful enough in determining the branch it was from.

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