CM-55207-Ruff fix

Author: mateusz-sterczewski

cycode/cli/files_collector/commit_range_documents.py

@@ -461,9 +461,8 @@ def parse_commit_range(commit_range: str, path: str) -> tuple[Optional[str], Opt
             if all_commits:
                 first_commit = all_commits.splitlines()[0]
                 return first_commit, head_commit, '..'
-            else:
-                logger.warning("No commits found for range '%s'", commit_range)
-                return None, None, None
+            logger.warning("No commits found for range '%s'", commit_range)
+            return None, None, None
         except Exception as e:
             logger.warning("Failed to parse commit range '%s'", commit_range, exc_info=e)
             return None, None, None

tests/cli/files_collector/test_commit_range_documents.py

@@ -891,14 +891,14 @@ def test_parse_all_for_empty_remote_scenario(self) -> None:
             test_file = os.path.join(temp_dir, 'test.py')
             with open(test_file, 'w') as f:
                 f.write("print('test')")
-            
+
             repo.index.add(['test.py'])
             commit = repo.index.commit('Initial commit')
-            
+
             # Test that '--all' (returned by calculate_pre_push_commit_range for empty remote)
             # can be parsed to a valid commit range
             parsed_from, parsed_to, separator = parse_commit_range('--all', temp_dir)
-            
+
             # Should return first commit to HEAD (which is the only commit in this case)
             assert parsed_from == commit.hexsha
             assert parsed_to == commit.hexsha
@@ -913,21 +913,21 @@ def test_parse_all_for_empty_remote_scenario_with_two_commits(self) -> None:
             test_file = os.path.join(temp_dir, 'test.py')
             with open(test_file, 'w') as f:
                 f.write("print('test')")
-            
+
             repo.index.add(['test.py'])
             commit1 = repo.index.commit('First commit')
-            
+
             # Create second commit
             test_file2 = os.path.join(temp_dir, 'test2.py')
             with open(test_file2, 'w') as f:
                 f.write("print('test2')")
-            
+
             repo.index.add(['test2.py'])
             commit2 = repo.index.commit('Second commit')
-            
+
             # Test that '--all' returns first commit to HEAD (second commit)
             parsed_from, parsed_to, separator = parse_commit_range('--all', temp_dir)
-            
+
             # Should return first commit to HEAD (second commit)
             assert parsed_from == commit1.hexsha  # First commit
             assert parsed_to == commit2.hexsha  # HEAD (second commit)
@@ -943,6 +943,7 @@ def test_parse_all_with_empty_repository_returns_none(self) -> None:
             assert parsed_to is None
             assert separator is None
 
+
 class TestParsePreReceiveInput:
     """Test the parse_pre_receive_input function with various pre-receive hook input scenarios."""