Merge pull request #55 from dupuy/avoid-markup-merge-conflict-fp
Don't report markup titles as merge conflicts
diff --git a/pre_commit_hooks/check_merge_conflict.py b/pre_commit_hooks/check_merge_conflict.py
index 5e274a3..f8bcadf 100644
--- a/pre_commit_hooks/check_merge_conflict.py
+++ b/pre_commit_hooks/check_merge_conflict.py
@@ -6,7 +6,8 @@
CONFLICT_PATTERNS = [
'<<<<<<< ',
- '=======',
+ '======= ',
+ '=======\n',
'>>>>>>> '
]
WARNING_MSG = 'Merge conflict string "{0}" found in {1}:{2}'
diff --git a/tests/check_merge_conflict_test.py b/tests/check_merge_conflict_test.py
index f3bd935..7e31313 100644
--- a/tests/check_merge_conflict_test.py
+++ b/tests/check_merge_conflict_test.py
@@ -47,7 +47,7 @@
@pytest.mark.parametrize(
- 'failing_contents', ('<<<<<<< HEAD', '=======', '>>>>>>> master'),
+ 'failing_contents', ('<<<<<<< HEAD\n', '=======\n', '>>>>>>> master\n'),
)
@pytest.mark.usefixtures('f1_is_a_conflict_file')
def test_merge_conflicts_failing(failing_contents):
@@ -56,7 +56,7 @@
@pytest.mark.parametrize(
- 'ok_contents', ('# <<<<<<< HEAD', '# =======', 'import my_module', ''),
+ 'ok_contents', ('# <<<<<<< HEAD\n', '# =======\n', 'import my_module', ''),
)
@pytest.mark.usefixtures('f1_is_a_conflict_file')
def test_merge_conflicts_ok(ok_contents):
@@ -67,5 +67,5 @@
@pytest.mark.usefixtures('in_tmpdir')
def test_does_not_care_when_not_in_a_conflict():
with io.open('README.md', 'w') as readme_file:
- readme_file.write('pre-commit\n=================\n')
+ readme_file.write('problem\n=======\n')
assert detect_merge_conflict(['README.md']) == 0