pre-commit/pre-commit-hooks
1
0
Fork 0
mirror of https://github.com/pre-commit/pre-commit-hooks.git synced 2026-03-29 18:16:52 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #55 from dupuy/avoid-markup-merge-conflict-fp

Browse source 
Don't report markup titles as merge conflicts
This commit is contained in:
Anthony Sottile 2015-05-08 20:38:17 -07:00
commit 16a4e68d88
2 changed files with 5 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

3
pre_commit_hooks/check_merge_conflict.py
View file

@ -6,7 +6,8 @@ import sys
CONFLICT_PATTERNS = [
'<<<<<<< ',
'=======',
'======= ',
'=======\n',
'>>>>>>> '
]
WARNING_MSG = 'Merge conflict string "{0}" found in {1}:{2}'

6
tests/check_merge_conflict_test.py
View file

@ -47,7 +47,7 @@ def f1_is_a_conflict_file(in_tmpdir):
@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 @@ def test_merge_conflicts_failing(failing_contents):
@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 @@ def test_merge_conflicts_ok(ok_contents):
@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