Skip to content

release: Bump to 0.2.3 and loosen redisvl dep #72

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jun 25, 2025
Merged

Conversation

tylerhutcherson
Copy link
Collaborator

No description provided.

@tylerhutcherson tylerhutcherson added the enhancement New feature or request label Jun 24, 2025
@tylerhutcherson tylerhutcherson changed the title release: Bump to 0.2.3 and loosen redisvl dependency release: Bump to 0.2.3 and loosen redisvl dep Jun 24, 2025
@abrookins abrookins requested a review from Copilot June 24, 2025 22:26
Copy link
Contributor

@Copilot 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 pull request focuses on updating the package version to 0.2.3 and loosening the dependency constraints for redisvl, while also removing several dependencies.

  • Bump version from 0.2.2 to 0.2.3.
  • Relax the redisvl dependency from "<0.8.0" to "<1.0.0".
  • Remove dependencies on numpy, python-ulid, tenacity, and the associated scipy configuration.
Comments suppressed due to low confidence (2)

libs/redis/pyproject.toml:16

  • Relaxing the redisvl version constraint to include versions up to 1.0.0 may allow incompatible changes; please verify that the consumers of this dependency have been tested against versions between 0.8.0 and 1.0.0.
redisvl = ">=0.6.0,<1.0.0"

libs/redis/pyproject.toml:15

  • The removal of numpy, python-ulid, and tenacity suggests these libraries are no longer needed; please confirm that this change does not affect runtime functionality and update any related migration documentation accordingly.
langchain-core = "^0.3"

Copy link
Collaborator

@abrookins abrookins left a comment

Choose a reason for hiding this comment

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

@tylerhutcherson This project uses python-ulid directly, so I'd keep the top-level dependency here rather than relying on python-ulid as a transitive dependency.

@tylerhutcherson
Copy link
Collaborator Author

@tylerhutcherson This project uses python-ulid directly, so I'd keep the top-level dependency here rather than relying on python-ulid as a transitive dependency.

Done! Good call

@tylerhutcherson tylerhutcherson merged commit 9315a41 into main Jun 25, 2025
12 checks passed
@tylerhutcherson tylerhutcherson deleted the release/0.2.3 branch June 25, 2025 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants