Conversation
|
Someone is attempting to deploy a commit to the deepset Team on Vercel. A member of the Team first needs to authorize it. |
|
Hi @Taranum01, thanks for your interest in contributing to Haystack! 🙏 ⛔ First-time contributors can have at most 1 open pull request in this repository until it has been approved, so this PR was closed automatically. Your open pull request #12991 is unaffected. Once it has been approved by a maintainer, you are welcome to open more PRs. Feel free to reopen this one at that point. See the contributing guidelines for details. This is an automated message to help us keep the review queue healthy. |
|
Taranum01 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
Summary
Fixes #12656.
The docstring on
PipelineBase._merge_super_component_pipelinesdescribed a function that does not exist in the source: it referenced parameters (parent_pipeline,child_components,merge_strategy,warn_on_conflict) and a return shape that don't match the actual implementation. This PR rewrites the docstring to match the real signature and behavior.No code changes; documentation only.
Changes
haystack/core/pipeline/base.py(around_merge_super_component_pipelines)Tests
No new tests added, per the maintainer note on the issue.