Skip to content

Commit 8bd3e66

Browse files
CM-55207-Fix tests further
1 parent 107e69b commit 8bd3e66

File tree

1 file changed

+2
-9
lines changed

1 file changed

+2
-9
lines changed

cycode/cli/files_collector/commit_range_documents.py

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -100,12 +100,7 @@ def collect_commit_range_diff_documents(
100100

101101
logger.debug(
102102
'Found all relevant files in commit %s',
103-
{
104-
'path': path,
105-
'commit_range': commit_range,
106-
'normalized_commit_range': normalized_commit_range,
107-
'commit_id': commit_id,
108-
},
103+
{'path': path, 'commit_range': normalized_commit_range, 'commit_id': commit_id},
109104
)
110105

111106
logger.debug('List of commit ids to scan, %s', {'commit_ids': commit_ids_to_scan})
@@ -450,12 +445,12 @@ def parse_commit_range(commit_range: str, path: str) -> tuple[Optional[str], Opt
450445
"""
451446
repo = git_proxy.get_repo(path)
452447

448+
separator = '..'
453449
if '...' in commit_range:
454450
from_spec, to_spec = commit_range.split('...', 1)
455451
separator = '...'
456452
elif '..' in commit_range:
457453
from_spec, to_spec = commit_range.split('..', 1)
458-
separator = '..'
459454
else:
460455
# Git commands like 'git diff <commit>' compare against HEAD.
461456
from_spec = commit_range
@@ -464,10 +459,8 @@ def parse_commit_range(commit_range: str, path: str) -> tuple[Optional[str], Opt
464459
# If a spec is empty (e.g., from '..master'), default it to 'HEAD'
465460
if not from_spec:
466461
from_spec = consts.GIT_HEAD_COMMIT_REV
467-
separator = '..'
468462
if not to_spec:
469463
to_spec = consts.GIT_HEAD_COMMIT_REV
470-
separator = '..'
471464

472465
try:
473466
# Use rev_parse to resolve each specifier to its full commit SHA

0 commit comments

Comments
 (0)