Skip to content

Index.conflictAdd should have nullable entries#997

Merged
implausible merged 1 commit into
masterfrom
make-conflict-add-nullable
Apr 18, 2016
Merged

Index.conflictAdd should have nullable entries#997
implausible merged 1 commit into
masterfrom
make-conflict-add-nullable

Conversation

@implausible

Copy link
Copy Markdown
Member

when you run into a conflict, sometimes one of the entries just plain old doesn't exist. An example being you create a file in 2 branches and try to merge them. They have no ancestor_entry because they were both created in their respective branches. Therefore, ancestor_entry should be null. The same can apply to our_entry and their_entry

@johnhaley81

Copy link
Copy Markdown
Collaborator

@implausible could you write a more descriptive summary of this commit please? these are all optional parameters doesn't give you any meaningful data. Maybe "Correct git_index_conflict_add optional parameters" or something?

when you run into a conflict, sometimes one of the entries just plain old doesn't exist. An example being you create a file in 2 branches and try to merge them. They have no ancestor_entry because they were both created in their respective branches. Therefore, ancestor_entry should be null. The same can apply to our_entry and their_entry
@implausible implausible force-pushed the make-conflict-add-nullable branch from ad72d9e to d6f0827 Compare April 18, 2016 19:16
@implausible implausible merged commit 0494943 into master Apr 18, 2016
@johnhaley81 johnhaley81 deleted the make-conflict-add-nullable branch April 18, 2016 19:17
implausible added a commit that referenced this pull request Apr 18, 2016
Index.conflictAdd should have nullable entries
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.

2 participants