Skip to content

Fix CI test job dependency install and align package modules with current test imports - #9

Merged
Khubaib8281 merged 3 commits into
mainfrom
copilot/fix-test-github-actions-job
Sep 7, 2026
Merged

Khubaib8281 merged 3 commits into
mainfrom
copilot/fix-test-github-actions-job

Conversation

Copilot AI commented Sep 5, 2026 •

Copy link
Copy Markdown
Contributor

The test GitHub Actions job failed because CI attempted to install requirements.txt, which no longer exists in this repo. This PR switches CI to install from pyproject.toml and restores the package modules currently referenced by the test suite.

  • CI dependency installation

    • Updated .github/workflows/ci.yml to install project + dev dependencies from pyproject.toml:
    - name: Install dependencies
      run: |
        python -m pip install --upgrade pip
        pip install ".[dev]"
  • Package module completeness for test collection

    • Added missing modules imported by existing tests:
      • docling_pk/preprocessor.py
      • docling_pk/ocr.py
      • docling_pk/parsers/__init__.py
      • docling_pk/parsers/cnic.py
      • docling_pk/parsers/matric.py
      • docling_pk/parsers/intermediate.py
      • docling_pk/parsers/_federal_board.py
    • Included minimal parser/preprocessing/OCR implementations so imports and expected code paths resolve under the current repository layout.

Co-authored-by: M-O-J-U <102480072+M-O-J-U@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job test Fix CI test job dependency install and align package modules with current test imports Sep 5, 2026
Copilot AI requested a review from M-O-J-U September 5, 2026 19:57
@Khubaib8281
Khubaib8281 marked this pull request as ready for review September 7, 2026 14:45
@Khubaib8281
Khubaib8281 merged commit e84a3cd into main Sep 7, 2026
1 check passed
@Khubaib8281
Khubaib8281 deleted the copilot/fix-test-github-actions-job branch September 7, 2026 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants