Skip to content
This repository was archived by the owner on Nov 13, 2024. It is now read-only.

Conversation

@izellevy
Copy link
Contributor

@izellevy izellevy commented Feb 26, 2024

Problem

Command/Nightly tokenizer changed the way of tokenization.

Solution

Changed the tests to reflect the recent version.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • Infrastructure change (CI configs, etc)
  • Non-code change (docs, etc)
  • None of the above: (explain here)

Test Plan

Describe specific steps for validating this change.

@izellevy izellevy requested a review from miararoy February 26, 2024 12:29
Copy link
Contributor

@miararoy miararoy left a comment

Choose a reason for hiding this comment

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

ltgm

@miararoy miararoy added this pull request to the merge queue Feb 26, 2024
Merged via the queue into pinecone-io:main with commit 580dc2c Feb 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants