Skip to content

[93] Remove pytest from dependencies#112

Open
tbazen wants to merge 1 commit into
alexzhang13:mainfrom
tbazen:93-pytest
Open

[93] Remove pytest from dependencies#112
tbazen wants to merge 1 commit into
alexzhang13:mainfrom
tbazen:93-pytest

Conversation

@tbazen

@tbazen tbazen commented Feb 18, 2026

Copy link
Copy Markdown

pytest is is removed from dependencies, because it is specified in test dependency group. This helps resolve pytest incompatibility issue:

#93

Copilot AI review requested due to automatic review settings February 18, 2026 17:35

Copilot AI left a comment

Copy link
Copy Markdown

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 resolves a dependency conflict issue (#93) by removing pytest from the main package dependencies, where it was incorrectly placed. The package rlms (Recursive Language Models) is a library for AI/ML applications and should not require pytest as a runtime dependency.

Changes:

  • Removed pytest>=9.0.2 from main dependencies in pyproject.toml
  • Updated uv.lock to reflect the removal of pytest from the package's runtime dependencies

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
pyproject.toml Removed pytest>=9.0.2 from the dependencies list (line 26 deleted)
uv.lock Updated lockfile to remove pytest from rlms package dependencies and requires-dist metadata

The change is correct and addresses the issue appropriately. pytest remains available in the test dependency group (pytest>=8.0.0 at line 56 of pyproject.toml), which is the proper location for testing dependencies. This allows developers to install test dependencies when needed using uv sync --group test, while preventing pytest version conflicts for downstream consumers of the package who install rlms as a dependency.

The existing CI/CD workflows (.github/workflows/test.yml) and Makefile already handle pytest installation separately, so no changes are needed there.


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

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