third-party/pre-commit
1
0
Fork 0
mirror of https://github.com/pre-commit/pre-commit.git synced 2026-02-17 08:14:42 +04:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #375 from vinayinvicible/master

Browse source 
only consider forward diff in changed files
This commit is contained in:
Anthony Sottile 2016-06-01 10:17:21 -07:00
commit d845ec6e36
2 changed files with 5 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
pre_commit/commands/run.py
View file

@ -51,7 +51,7 @@ def _print_user_skipped(hook, write, args):
def get_changed_files(new, old): def get_changed_files(new, old):
return cmd_output( return cmd_output(
'git', 'diff', '--name-only', '{0}..{1}'.format(old, new), 'git', 'diff', '--name-only', '{0}...{1}'.format(old, new),
)[1].splitlines() )[1].splitlines()

4
tests/commands/run_test.py
View file

@ -401,6 +401,10 @@ def test_get_changed_files():
) )
assert files == ['CHANGELOG.md', 'setup.py'] assert files == ['CHANGELOG.md', 'setup.py']
# files changed in source but not in origin should not be returned
files = get_changed_files('HEAD~10', 'HEAD')
assert files == []
def test_lots_of_files(mock_out_store_directory, tempdir_factory): def test_lots_of_files(mock_out_store_directory, tempdir_factory):
# windows xargs seems to have a bug, here's a regression test for # windows xargs seems to have a bug, here's a regression test for