Skip to content

docs: adding docs for ElasticSearchHybridRetriever#11806

Merged
davidsbatista merged 7 commits into
mainfrom
docs/adding-docs-for-ElasticSearchHybridRetriever
Jun 30, 2026
Merged

docs: adding docs for ElasticSearchHybridRetriever#11806
davidsbatista merged 7 commits into
mainfrom
docs/adding-docs-for-ElasticSearchHybridRetriever

Conversation

@davidsbatista

Copy link
Copy Markdown
Contributor

Related Issues

Checklist

  • I have read the contributors guidelines and the code of conduct.
  • I have updated the related issue with new insights and changes.
  • I have added unit tests and updated the docstrings.
  • I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test: and added ! in case the PR includes breaking changes.
  • I have documented my code.
  • I have added a release note file, following the contributors guidelines.
  • I have run pre-commit hooks and fixed any issue.

@vercel

vercel Bot commented Jun 29, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
haystack-docs Ready Ready Preview, Comment Jun 30, 2026 10:22am

Request Review

@davidsbatista davidsbatista added the ignore-for-release-notes PRs with this flag won't be included in the release notes. label Jun 29, 2026
@davidsbatista davidsbatista marked this pull request as ready for review June 29, 2026 11:36
@davidsbatista davidsbatista requested a review from a team as a code owner June 29, 2026 11:37
@davidsbatista davidsbatista requested review from bogdankostic and removed request for a team June 29, 2026 11:37
@claude

claude Bot commented Jun 29, 2026

Copy link
Copy Markdown

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@bogdankostic bogdankostic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good in general, just a few minor comments.

Comment thread docs-website/docs/pipeline-components/retrievers/elasticsearchhybridretriever.mdx Outdated
Comment thread docs-website/docs/pipeline-components/retrievers/elasticsearchhybridretriever.mdx Outdated
Comment thread docs-website/docs/pipeline-components/retrievers/elasticsearchhybridretriever.mdx Outdated
Comment thread docs-website/docs/pipeline-components/retrievers/elasticsearchhybridretriever.mdx Outdated

@bogdankostic bogdankostic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's apply the changes also here before merging.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

good catch, thanks 👍🏽

@davidsbatista davidsbatista merged commit 0a83580 into main Jun 30, 2026
23 checks passed
@davidsbatista davidsbatista deleted the docs/adding-docs-for-ElasticSearchHybridRetriever branch June 30, 2026 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ignore-for-release-notes PRs with this flag won't be included in the release notes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add docs page for ElasticsearchHybridRetriever component

2 participants