Skip to content

Commit 6d92ed9

Browse files
refactor: refactor module names to avoid function/module name clashes (#114)
- Renamed: - `clone.py` → `repository_clone.py` - `ingest.py` → `repository_ingest.py` - `ingest_from_query.py` → `query_ingestion.py` - `parse_query.py` → `query_parser.py`
1 parent 3af99bf commit 6d92ed9

File tree

12 files changed

+36
-36
lines changed

12 files changed

+36
-36
lines changed

src/gitingest/__init__.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
""" Gitingest: A package for ingesting data from git repositories. """
22

3-
from gitingest.clone import clone_repo
4-
from gitingest.ingest import ingest
5-
from gitingest.ingest_from_query import run_ingest_query
6-
from gitingest.parse_query import parse_query
3+
from gitingest.query_ingestion import run_ingest_query
4+
from gitingest.query_parser import parse_query
5+
from gitingest.repository_clone import clone_repo
6+
from gitingest.repository_ingest import ingest
77

88
__all__ = ["run_ingest_query", "clone_repo", "parse_query", "ingest"]

src/gitingest/cli.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@
44

55
import click
66

7-
from gitingest.ingest import ingest
8-
from gitingest.ingest_from_query import MAX_FILE_SIZE
7+
from gitingest.query_ingestion import MAX_FILE_SIZE
8+
from gitingest.repository_ingest import ingest
99

1010

1111
@click.command()
File renamed without changes.
File renamed without changes.
File renamed without changes.

src/gitingest/ingest.py renamed to src/gitingest/repository_ingest.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,9 @@
55
import shutil
66

77
from config import TMP_BASE_PATH
8-
from gitingest.clone import CloneConfig, clone_repo
9-
from gitingest.ingest_from_query import run_ingest_query
10-
from gitingest.parse_query import parse_query
8+
from gitingest.query_ingestion import run_ingest_query
9+
from gitingest.query_parser import parse_query
10+
from gitingest.repository_clone import CloneConfig, clone_repo
1111

1212

1313
def ingest(

src/process_query.py renamed to src/query_processor.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,9 @@
77
from starlette.templating import _TemplateResponse
88

99
from config import EXAMPLE_REPOS, MAX_DISPLAY_SIZE
10-
from gitingest.clone import CloneConfig, clone_repo
11-
from gitingest.ingest_from_query import run_ingest_query
12-
from gitingest.parse_query import parse_query
10+
from gitingest.query_ingestion import run_ingest_query
11+
from gitingest.query_parser import parse_query
12+
from gitingest.repository_clone import CloneConfig, clone_repo
1313
from server_utils import Colors, log_slider_to_size
1414

1515
templates = Jinja2Templates(directory="templates")

src/routers/dynamic.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
from fastapi.responses import HTMLResponse
55
from fastapi.templating import Jinja2Templates
66

7-
from process_query import process_query
7+
from query_processor import process_query
88
from server_utils import limiter
99

1010
router = APIRouter()

src/routers/index.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
from fastapi.templating import Jinja2Templates
66

77
from config import EXAMPLE_REPOS
8-
from process_query import process_query
8+
from query_processor import process_query
99
from server_utils import limiter
1010

1111
router = APIRouter()

tests/test_ingest.py renamed to tests/test_query_ingestion.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
1-
""" Tests for the ingest_from_query module """
1+
""" Tests for the query_ingestion module """
22

33
from pathlib import Path
44
from typing import Any
55
from unittest.mock import patch
66

7-
from gitingest.ingest_from_query import _extract_files_content, _read_file_content, _scan_directory
7+
from gitingest.query_ingestion import _extract_files_content, _read_file_content, _scan_directory
88

99

1010
def test_scan_directory(temp_directory: Path, sample_query: dict[str, Any]) -> None:
@@ -42,8 +42,8 @@ def test_read_file_content_with_notebook(tmp_path: Path):
4242
notebook_path = tmp_path / "dummy_notebook.ipynb"
4343
notebook_path.write_text("{}", encoding="utf-8") # minimal JSON
4444

45-
# Patch the symbol as it is used in ingest_from_query
46-
with patch("gitingest.ingest_from_query.process_notebook") as mock_process:
45+
# Patch the symbol as it is used in query_ingestion
46+
with patch("gitingest.query_ingestion.process_notebook") as mock_process:
4747
_read_file_content(notebook_path)
4848
mock_process.assert_called_once_with(notebook_path)
4949

@@ -52,7 +52,7 @@ def test_read_file_content_with_non_notebook(tmp_path: Path):
5252
py_file_path = tmp_path / "dummy_file.py"
5353
py_file_path.write_text("print('Hello')", encoding="utf-8")
5454

55-
with patch("gitingest.ingest_from_query.process_notebook") as mock_process:
55+
with patch("gitingest.query_ingestion.process_notebook") as mock_process:
5656
_read_file_content(py_file_path)
5757
mock_process.assert_not_called()
5858

0 commit comments

Comments
 (0)