Skip to content

Conversation

@vladvildanov
Copy link
Contributor

Feature is consider experimental as interface and response structure might be flacky

@coveralls
Copy link

coveralls commented Nov 9, 2025

Coverage Status

coverage: 92.729% (-0.5%) from 93.185%
when pulling f4f61a7 on vv-hybrid-search
into f4dcf6c on main.

@tillkruss
Copy link
Member

So many loc 😅

@vladvildanov
Copy link
Contributor Author

@tillkruss Yeah, but most of it is tests cause API is quite extensive 😄 Coverage decrease is actually wrong, all of the methods is tested within integration tests

@vladvildanov
Copy link
Contributor Author

@tillkruss You're right, let's mark it experimental in logs as well

@vladvildanov vladvildanov changed the title Added support for FT.HYBRID Added experimental support for FT.HYBRID Nov 11, 2025
Copy link
Contributor

@htemelski-oss htemelski-oss left a comment

Choose a reason for hiding this comment

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

Looks good

@vladvildanov vladvildanov merged commit f11e855 into main Nov 11, 2025
32 of 33 checks passed
@vladvildanov vladvildanov deleted the vv-hybrid-search branch November 11, 2025 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

5 participants