Skip to content

fix(ci): install pytest for python sdk omnibus test#144

Merged
nullure merged 4 commits intoCaviraOSS:mainfrom
stevo1403:fix/dashboard-ui-profile-from-main
Feb 26, 2026
Merged

fix(ci): install pytest for python sdk omnibus test#144
nullure merged 4 commits intoCaviraOSS:mainfrom
stevo1403:fix/dashboard-ui-profile-from-main

Conversation

@stevo1403
Copy link
Contributor

@stevo1403 stevo1403 commented Feb 24, 2026

Summary

  • install pytest in the Python SDK CI job
  • run tests via python -m pytest for consistent invocation

Why

The Test Python SDK check fails in CI with pytest: command not found because packages/openmemory-py does not define a [dev] extra in pyproject.toml, so pip install -e .[dev] does not install pytest.

Validation

  • Verified failure in run 22349964921 (Run Omnibus Test (Deep Verify) step).
  • This PR updates .github/workflows/ci.yml only.

Copilot AI review requested due to automatic review settings February 24, 2026 12:10
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes the Python SDK CI job by ensuring pytest and its dependencies are installed before running the test suite. The pyproject.toml for the Python SDK doesn't define a [project.optional-dependencies] section with a dev extra, so the pip install -e .[dev] command was not installing pytest, causing test failures.

Changes:

  • Added explicit pip install pytest to ensure pytest is available for test execution
  • Changed test invocation from pytest to python -m pytest for consistent Python environment usage

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@nullure nullure merged commit 56dda38 into CaviraOSS:main Feb 26, 2026
3 checks passed
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