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

Doc updates and change default g:conflict_marker_common_ancestors #20

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Frederick888
Copy link

@Frederick888 Frederick888 commented Feb 17, 2022

  • Fix default g:conflict_marker_common_ancestors

  • Match 7 or more pipes as common ancestors marker

    I'm not that sure about this commit. From [1]'s commit message, it
    seems that there can be more than 7 pipes when conflict is 'nested'.
    However I've never seen such a case and I wasn't able to find an
    example online.

    But if my assumption is correct, I guess it makes more sense to
    match 7 or more pipes to avoid partial highlight?

  • Add full line common ancestors example

    Since the goal of the example configuration here seems to be making
    highlight of each whole marker line consistent, it'd be nice to
    configure g:conflict_marker_common_ancestors as well to prevent
    ancestor commit hashes from being highlighted differently.

    Before: image

    After: image

[1] 0ada2b9

The default value was changed in [1].

[1] rhysd@0ada2b9
@Frederick888 Frederick888 marked this pull request as ready for review February 17, 2022 13:40
Since the goal of the example configuration here seems to be making
highlight of each whole marker line consistent, it'd be nice to
configure g:conflict_marker_common_ancestors as well to prevent ancestor
commit hashes from being highlighted differently.
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