Skip to content

docs: hide docling remote VLM component from main#10490

Merged
mendonk merged 1 commit into
mainfrom
docs-hide-docling-vlm
Nov 3, 2025
Merged

docs: hide docling remote VLM component from main#10490
mendonk merged 1 commit into
mainfrom
docs-hide-docling-vlm

Conversation

@mendonk
Copy link
Copy Markdown
Collaborator

@mendonk mendonk commented Nov 3, 2025

Hide component from main docs build.
Will be added back for 1.7 #10489

Summary by CodeRabbit

  • Documentation
    • Removed documentation for the Docling VLM pipeline with remote model configuration, including provider-specific setup guidance. Other Docling components remain available.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Nov 3, 2025

Walkthrough

Documentation update removing the Docling VLM pipeline with remote model section, including descriptive content and parameter tables for IBM Cloud and OpenAI-Compatible providers, from the bundles-docling component documentation file.

Changes

Cohort / File(s) Summary
Docling VLM Pipeline Documentation Removal
docs/docs/Components/bundles-docling.mdx
Removed entire Docling VLM pipeline with remote model section, including section header, descriptive paragraphs, and parameter tables (files, provider, vlm_prompt, watsonx_api_key, watsonx_project_id, url, model_name, openai_base_url, openai_api_key, openai_model) for IBM Cloud and OpenAI-Compatible providers.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

enhancement

Suggested reviewers

  • erichare

Pre-merge checks and finishing touches

Important

Pre-merge checks failed

Please resolve all errors before merging. Addressing warnings is optional.

❌ Failed checks (3 inconclusive)
Check name Status Explanation Resolution
Test Coverage For New Implementations ❓ Inconclusive This pull request exclusively modifies documentation files (docs/docs/Components/bundles-docling.mdx) by removing component documentation content. The PR summary explicitly states 'Alterations to the declarations of exported or public entities: None,' confirming no code changes, new features, or functionality modifications are present. This is purely a documentation maintenance task, not an implementation of new code or features that would require test coverage verification. The test coverage check is not applicable to documentation-only changes. This check should be marked as 'Not Applicable' for PRs that exclusively modify documentation without any code implementation, new components, features, or bug fixes. Documentation maintenance tasks do not require test coverage verification.
Test Quality And Coverage ❓ Inconclusive The 'Test Quality and Coverage' check is designed to evaluate tests for new code implementations, async functions, API endpoints, and behavioral validation. This pull request is exclusively a documentation change that removes the 'Docling Remote VLM' section from the bundles-docling.mdx file. No code files were modified, no implementation changes were made, and no test files were created or modified. This check is not applicable to documentation-only changes. The 'Test Quality and Coverage' check should only be applied to pull requests that introduce or modify functional code implementations. For this PR, consider applying documentation-specific quality checks such as accuracy, completeness, formatting, or consistency validation instead of code-focused testing checks.
Test File Naming And Structure ❓ Inconclusive This pull request is a documentation-only change that removes content from docs/docs/Components/bundles-docling.mdx. No test files (.py, .ts, .tsx, or .test files) are modified or introduced in this commit. The custom check for test file naming and structure patterns cannot be evaluated because there are no test files present in this PR. The test file naming and structure verification is not applicable to documentation-only changes. This check should only be applied when test files are actually added or modified in a pull request. For future PRs involving code or test changes, verify that test files follow naming patterns such as test_*.py or *.test.ts and maintain appropriate directory structure.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title 'docs: hide docling remote VLM component from main' directly and clearly describes the main change in the changeset. The raw summary confirms that the entire Docling VLM pipeline remote model documentation has been removed from the docs, and the PR objectives explicitly state the intent is to hide the component from the main docs build. The title is specific, concise, and accurately reflects the primary modification.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch docs-hide-docling-vlm

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2207f51 and 526aea3.

📒 Files selected for processing (1)
  • docs/docs/Components/bundles-docling.mdx (0 hunks)
💤 Files with no reviewable changes (1)
  • docs/docs/Components/bundles-docling.mdx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: Test Docs Build / Test Docs Build
  • GitHub Check: build-and-deploy

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Nov 3, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 3, 2025

Build successful! ✅
Deploying docs draft.
Deploy successful! View draft

@github-actions github-actions Bot added documentation Improvements or additions to documentation and removed documentation Improvements or additions to documentation labels Nov 3, 2025
@github-actions github-actions Bot added the lgtm This PR has been approved by a maintainer label Nov 3, 2025
@mendonk mendonk added this pull request to the merge queue Nov 3, 2025
Merged via the queue into main with commit 9613d4e Nov 3, 2025
24 checks passed
@mendonk mendonk deleted the docs-hide-docling-vlm branch November 3, 2025 17:09
korenLazar pushed a commit to kiran-kate/langflow that referenced this pull request Nov 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation lgtm This PR has been approved by a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants