mirror of
https://github.com/pre-commit/pre-commit.git
synced 2026-02-19 17:14:43 +04:00
Apply suggestions from code review
This commit is contained in:
parent
e7959ecc5d
commit
6c87f31a79
1 changed files with 5 additions and 2 deletions
|
|
@ -114,7 +114,9 @@ def _rev_exists(rev: str) -> bool:
|
||||||
|
|
||||||
|
|
||||||
def _is_ancestor(ancestor: str, rev: str):
|
def _is_ancestor(ancestor: str, rev: str):
|
||||||
return not subprocess.call(('git', 'merge-base', '--is-ancestor', ancestor, rev))
|
return not subprocess.call(
|
||||||
|
('git', 'merge-base', '--is-ancestor', ancestor, rev)
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
def _pre_push_ns(
|
def _pre_push_ns(
|
||||||
|
|
@ -130,7 +132,8 @@ def _pre_push_ns(
|
||||||
local_branch, local_sha, remote_branch, remote_sha = parts
|
local_branch, local_sha, remote_branch, remote_sha = parts
|
||||||
if local_sha == Z40:
|
if local_sha == Z40:
|
||||||
continue
|
continue
|
||||||
elif remote_sha != Z40 and _rev_exists(remote_sha) and _is_ancestor(remote_sha, local_sha):
|
elif remote_sha != Z40 and _rev_exists(remote_sha) \
|
||||||
|
and _is_ancestor(remote_sha, local_sha):
|
||||||
return _ns(
|
return _ns(
|
||||||
'pre-push', color,
|
'pre-push', color,
|
||||||
from_ref=remote_sha, to_ref=local_sha,
|
from_ref=remote_sha, to_ref=local_sha,
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue