Skip to content

docs: remove www datastax links#10249

Merged
mendonk merged 3 commits into
mainfrom
docs-datastax-com-links
Oct 13, 2025
Merged

docs: remove www datastax links#10249
mendonk merged 3 commits into
mainfrom
docs-datastax-com-links

Conversation

@mendonk
Copy link
Copy Markdown
Collaborator

@mendonk mendonk commented Oct 13, 2025

Remove links in main/docs to www.datastax.com

Summary by CodeRabbit

  • Documentation
    • Updated the development setup guidance to recommend installing Langflow Desktop for a simpler, faster onboarding experience.
    • Refreshed the “Chat with RAG” tutorial prerequisites by converting linked references to plain text mentions of vector databases and RAG, improving readability and consistency.
    • Clarified messaging and reduced distractions in docs; no functional changes to features or workflows.

@mendonk mendonk self-assigned this Oct 13, 2025
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Oct 13, 2025

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

Updates limited to documentation text: the devcontainer README now references Langflow Desktop installation, and the “chat-with-rag” tutorial adjusts prerequisite wording by removing external links. No code, logic, or control flow changes.

Changes

Cohort / File(s) Summary
Devcontainer README update
./.devcontainer/README.md
Replaced promotional line with instruction to download and install Langflow Desktop. Messaging-only change.
RAG tutorial prerequisites tweak
docs/docs/Tutorials/chat-with-rag.mdx
Removed hyperlink formatting for “vector databases and RAG” in prerequisites; kept plain text mentions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

documentation, size:L, lgtm

Suggested reviewers

  • aimurphy
  • edwinjosechittilappilly
  • melienherrera

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Test File Naming And Structure ⚠️ Warning After inspecting the repository, I found numerous backend tests not following the required test_*.py naming convention and several frontend tests whose filenames end with .spec.ts or .spec.tsx instead of the mandated .test.ts or .test.tsx. Because existing test files violate the prescribed naming patterns, the repository does not satisfy the Test File Naming and Structure check. Please rename backend tests so they use the test_*.py pattern and update all frontend spec files to the .test.ts or .test.tsx convention, ensuring future additions follow these rules to achieve compliance.
✅ Passed checks (6 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title succinctly and accurately describes the primary change of removing DataStax links from the documentation without introducing unnecessary detail, and it follows the conventional “docs:” prefix pattern for clarity.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
Test Coverage For New Implementations ✅ Passed No application code or functionality changes were introduced in this PR; it merely updates documentation files, so no new or modified tests are required to cover new behavior, and the absence of test changes is acceptable under the custom check criteria.
Test Quality And Coverage ✅ Passed The PR only adjusts documentation messaging without modifying code, APIs, or behavior, so there is no new functionality requiring test coverage. Given the strict focus on docs, existing test suites remain adequate and no additional tests are warranted for this change.
Excessive Mock Usage Warning ✅ Passed The custom check targets excessive mocking in test files, but this pull request only updates documentation (.devcontainer/README.md and docs/docs/Tutorials/chat-with-rag.mdx) and introduces no changes to any tests or mocks. Because no test code is touched, there is no evidence of mock overuse or related concerns within the scope of this PR, so the check passes by default.

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 Oct 13, 2025
@github-actions

This comment has been minimized.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 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 f24a064 and 8490dd6.

📒 Files selected for processing (2)
  • .devcontainer/README.md (1 hunks)
  • docs/docs/Tutorials/chat-with-rag.mdx (1 hunks)
🧰 Additional context used
📓 Path-based instructions (2)
docs/**/*.{md,mdx}

📄 CodeRabbit inference engine (.cursor/rules/docs_development.mdc)

docs/**/*.{md,mdx}: All Markdown/MDX pages must start with front matter including at least title and description; include sidebar_position for docs pages when applicable
Code blocks must specify a language and may include a title (```lang title="…")
Use sentence case for headings and keep paragraphs short and scannable
Write in second person, present tense, with a professional but approachable tone
Use inline code with backticks for code terms; use bold for UI elements and italics for emphasis; keep lists in parallel structure
Ensure internal links are functional and navigation works (update cross-references as needed)
Verify all code examples in docs and blog actually run as shown
Use correct terminology capitalization: Langflow, Component, Flow, API, JSON
Reference images with absolute paths under /img/... and provide descriptive alt text

Files:

  • docs/docs/Tutorials/chat-with-rag.mdx
docs/docs/**/*.{md,mdx}

📄 CodeRabbit inference engine (.cursor/rules/docs_development.mdc)

Use Docusaurus admonitions (:::+tip|warning|danger) instead of custom callouts in docs pages

Files:

  • docs/docs/Tutorials/chat-with-rag.mdx
🪛 markdownlint-cli2 (0.18.1)
.devcontainer/README.md

5-5: Link fragments should be valid

(MD051, link-fragments)

⏰ 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
🔇 Additional comments (1)
docs/docs/Tutorials/chat-with-rag.mdx (1)

18-18: Clean removal of external link.

Removing the external link keeps the prerequisite concise while still conveying the requirement. Change looks good.

Comment thread .devcontainer/README.md Outdated
@github-actions github-actions Bot added documentation Improvements or additions to documentation and removed documentation Improvements or additions to documentation labels Oct 13, 2025
@mendonk mendonk requested a review from aimurphy October 13, 2025 15:48
@github-actions github-actions Bot added the lgtm This PR has been approved by a maintainer label Oct 13, 2025
@mendonk mendonk enabled auto-merge October 13, 2025 16:43
@github-actions github-actions Bot added documentation Improvements or additions to documentation and removed documentation Improvements or additions to documentation labels Oct 13, 2025
@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 13, 2025

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

@mendonk mendonk added this pull request to the merge queue Oct 13, 2025
Merged via the queue into main with commit 385c859 Oct 13, 2025
18 checks passed
@mendonk mendonk deleted the docs-datastax-com-links branch October 13, 2025 17:03
korenLazar pushed a commit to kiran-kate/langflow that referenced this pull request Nov 11, 2025
* devcontainer

* chat-with-rag-links

* full-docs-link
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