Skip to content
This repository has been archived by the owner on Nov 8, 2017. It is now read-only.

Add three-way merge markers support #219

Merged
merged 1 commit into from
Apr 4, 2016

Conversation

ypresto
Copy link
Collaborator

@ypresto ypresto commented Apr 4, 2016

Closes #9

These are my changes
||||||| merged common ancestors
<<<<<<< Temporary merge branch 1
Refer: http://stackoverflow.com/questions/16990657/git-merge-diff3-style-need-explanation
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

very complex format 😕

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, yeah, that can get kind of crazy. I wonder if it can nest arbitrarily with multiple criss-cross merges... ? In that case we'd probably need some kind of recursive descent parser just to identify marked conflicts in the first place.

@smashwilson
Copy link
Owner

!!!

That's awesome!

@ypresto
Copy link
Collaborator Author

ypresto commented Apr 4, 2016

I did it!!! 😃

@smashwilson smashwilson merged commit 3c01b0f into smashwilson:master Apr 4, 2016
@ypresto ypresto deleted the three-way branch April 4, 2016 17:55
@smashwilson
Copy link
Owner

~/code/merge-conflicts docker:dev (master=) 
$ apm publish minor
Preparing and tagging a new version ✓
Pushing v1.4.0 tag ✓
Publishing merge-conflicts@v1.4.0 ✓

@varemenos
Copy link

thank you very much for this!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants