Skip to content

Fix test-and-eval workflow failure by installing test dependencies#1

Merged
murchu27 merged 2 commits into
mainfrom
copilot/fix-test-and-eval-job
Jun 6, 2026
Merged

Fix test-and-eval workflow failure by installing test dependencies#1
murchu27 merged 2 commits into
mainfrom
copilot/fix-test-and-eval-job

Conversation

Copilot AI commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

The test-and-eval GitHub Actions job failed before evaluation because pytest was unavailable in CI. The workflow installed only runtime dependencies, but tests require the repository’s dev extras.

  • Root cause

    • CI dependency install used pip install ., which excludes optional dev packages.
    • pytest is defined under [project.optional-dependencies].dev, so the test step failed with command not found.
  • Workflow update

    • Updated .github/workflows/ci.yml to install package with dev extras in the Install dependencies step.
    • This aligns CI setup with the workflow’s actual execution path (pytest -q).
- pip install .
+ pip install ".[dev]"

Copilot AI changed the title [WIP] Fix failing GitHub Actions job test-and-eval Fix test-and-eval workflow failure by installing test dependencies Jun 6, 2026
Copilot AI requested a review from murchu27 June 6, 2026 19:02

@murchu27 murchu27 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updates ci.yml to properly install the dependencies it needs to run end-to-end. Specifically, it now installs pytest so that it can run unit tests. CI workflow now succeeds, so this is good to go.

@murchu27 murchu27 marked this pull request as ready for review June 6, 2026 19:09
@murchu27 murchu27 merged commit 58e5aa2 into main Jun 6, 2026
1 check passed
@murchu27 murchu27 deleted the copilot/fix-test-and-eval-job branch June 6, 2026 19:09
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.

2 participants