-
Notifications
You must be signed in to change notification settings - Fork 886
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
Move FindMergeBase() overloads to ObjectDatabase #951
Comments
I think this issue and #864 are the same thing. They both have the up for grabs tag, so would it be cool if I take a look at this? This looks like it should be pretty easy to revise, and I've been wanting to find a good entry point to contributing! |
I went ahead and gave it a try. If it's something you're ok with me doing, I'll open a PR for it. If not, well no big deal, I still got to play around in the codebase a bit! |
@bording Thanks and welcome to the project! I think it would be great if this was something you wanted to take a look at! I haven't looked at the changes in detail, but if you wanted to open a PR, that would make it easier to proceed. Thanks! |
Fixed by #957 |
IQueryableCommitLog.FindMergeBase()
CommitLog.FindMergeBase()
implementation toObjectDatabase.FindMergeBase()
CommitLog.FindMergeBase()
delegate toObjectDatabase.FindMergeBase()
CommitLog.FindMergeBase()
The text was updated successfully, but these errors were encountered: