Skip to content

Add CI workflow to run automated tests#5

Merged
emredeveloper merged 2 commits intomainfrom
codex/provide-example-for-description-section-qcbky3
Oct 20, 2025
Merged

Add CI workflow to run automated tests#5
emredeveloper merged 2 commits intomainfrom
codex/provide-example-for-description-section-qcbky3

Conversation

@emredeveloper
Copy link
Copy Markdown
Owner

Summary

  • add a GitHub Actions workflow that installs development dependencies and runs the pytest suite
  • document the new continuous integration workflow in the top-level README

Testing

  • not run (network restrictions prevented installing dependencies)

https://chatgpt.com/codex/tasks/task_e_68f642ed1adc832fbf832cf1c9f9331c

@emredeveloper emredeveloper merged commit 99473d6 into main Oct 20, 2025
1 check failed
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/ci.yml
Comment on lines +25 to +31
- name: Install dependencies
run: |
pip install -r "Memory LLM/requirements-dev.txt"

- name: Run test suite
run: |
python -m pytest -c "Memory LLM/pyproject.toml"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Install project before running tests in CI

The workflow installs only requirements-dev.txt and then runs python -m pytest, but the package lives under Memory LLM/mem_llm and is not on PYTHONPATH in a fresh runner unless it is installed. As a result, the tests will fail immediately with ModuleNotFoundError: mem_llm instead of exercising the suite. Add a step such as pip install -e "Memory LLM" or change into the package directory before invoking pytest so the module can be imported.

Useful? React with 👍 / 👎.

@emredeveloper emredeveloper deleted the codex/provide-example-for-description-section-qcbky3 branch January 20, 2026 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant