Skip to content

Conversation

@msimmons82
Copy link

@msimmons82 msimmons82 commented Nov 24, 2025

Summary by CodeRabbit

  • Documentation
    • Updated product references to reflect current naming conventions.
    • Added links to additional product information and support contact details.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Nov 24, 2025

Walkthrough

Updated the DDS Shapes Subscriber README to reference RTI Connext instead of RTI Connext Express. Added a link to additional Connext information and a contact email. Minor presentation adjustments with no functional impact.

Changes

Cohort / File(s) Summary
Documentation Updates
operators/dds/dds_shapes_subscriber/README.md
Updated product naming from RTI Connext Express to RTI Connext; added reference link and contact email; minor wording and presentation changes

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • Single documentation file with no code or functional changes
  • Content updates limited to product naming alignment and informational additions

Pre-merge checks

✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly references the main change: updating the README for RTI Connext (replacing RTI Connext Express). It is specific and clearly summarizes the primary modification.

Tip

📝 Customizable high-level summaries are now available in beta!

You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.

  • Provide your own instructions using the high_level_summary_instructions setting.
  • Format the summary however you like (bullet lists, tables, multi-section layouts, contributor stats, etc.).
  • Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.

Example instruction:

"Divide the high-level summary into five sections:

  1. 📝 Description — Summarize the main change in 50–60 words, explaining what was done.
  2. 📓 References — List relevant issues, discussions, documentation, or related PRs.
  3. 📦 Dependencies & Requirements — Mention any new/updated dependencies, environment variable changes, or configuration updates.
  4. 📊 Contributor Summary — Include a Markdown table showing contributions:
    | Contributor | Lines Added | Lines Removed | Files Changed |
  5. ✔️ Additional Notes — Add any extra reviewer context.
    Keep each section concise (under 200 words) and use bullet or numbered lists for clarity."

Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later.


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.

Copy link

@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: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 38cd23d and 9cdd70a.

📒 Files selected for processing (1)
  • operators/dds/dds_shapes_subscriber/README.md (1 hunks)
🧰 Additional context used
📓 Path-based instructions (2)
{operators,applications,workflows,gxf_extensions,tutorials}/**/README.md

📄 CodeRabbit inference engine (CONTRIBUTING.md)

README.md must include purpose, usage instructions, requirements, examples, and architecture overview

Files:

  • operators/dds/dds_shapes_subscriber/README.md
**/*.md

📄 CodeRabbit inference engine (CONTRIBUTING.md)

Use Holoscan SDK glossary terms when referring to HoloHub-specific locations in documentation

Files:

  • operators/dds/dds_shapes_subscriber/README.md
🪛 markdownlint-cli2 (0.18.1)
operators/dds/dds_shapes_subscriber/README.md

8-8: Bare URL used

(MD034, no-bare-urls)


8-8: Bare URL used

(MD034, no-bare-urls)


8-8: Link text should be descriptive

(MD059, descriptive-link-text)

to downstream operators.

This operator requires an installation of [RTI Connext Express](https://content.rti.com/l/983311/2025-07-08/q5x1n8) to provide access to the DDS domain, as specified by the [OMG Data-Distribution Service](https://www.omg.org/omg-dds-portal/). To obtain a license/activation key, please [click here](https://content.rti.com/l/983311/2025-07-25/q6729c). Please see the [usage rules](https://www.rti.com/products/connext-express) for Connext Express.
This operator requires an installation of [RTI Connext](https://content.rti.com/l/983311/2025-07-08/q5x1n8) to provide access to the DDS domain, as specified by the [OMG Data-Distribution Service](https://www.omg.org/omg-dds-portal/). To obtain a license/activation key, please [click here](https://content.rti.com/l/983311/2025-07-25/q6729c). Additional information on Connext can be found here: https://www.rti.com/products/third-party-integrations/nvidia. For any questions, please email [email protected]
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Fix bare URLs and improve link text clarity.

Line 8 contains bare URLs and non-descriptive link text that violate markdown best practices:

  • The bare URL https://www.rti.com/products/third-party-integrations/nvidia should be wrapped in a markdown link with descriptive text
  • The link text [click here] is non-descriptive; use text that conveys the destination's purpose (e.g., "obtain a license/activation key")
  • The email [email protected] should be formatted as a mailto link for consistency

Apply this diff to fix the issues:

-This operator requires an installation of [RTI Connext](https://content.rti.com/l/983311/2025-07-08/q5x1n8) to provide access to the DDS domain, as specified by the [OMG Data-Distribution Service](https://www.omg.org/omg-dds-portal/). To obtain a license/activation key, please [click here](https://content.rti.com/l/983311/2025-07-25/q6729c). Additional information on Connext can be found here: https://www.rti.com/products/third-party-integrations/nvidia.  For any questions, please email [email protected]
+This operator requires an installation of [RTI Connext](https://content.rti.com/l/983311/2025-07-08/q5x1n8) to provide access to the DDS domain, as specified by the [OMG Data-Distribution Service](https://www.omg.org/omg-dds-portal/). To obtain a license/activation key, please [request a license/activation key](https://content.rti.com/l/983311/2025-07-25/q6729c). Additional information on Connext can be found [here](https://www.rti.com/products/third-party-integrations/nvidia). For any questions, please email [[email protected]](mailto:[email protected])
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
This operator requires an installation of [RTI Connext](https://content.rti.com/l/983311/2025-07-08/q5x1n8) to provide access to the DDS domain, as specified by the [OMG Data-Distribution Service](https://www.omg.org/omg-dds-portal/). To obtain a license/activation key, please [click here](https://content.rti.com/l/983311/2025-07-25/q6729c). Additional information on Connext can be found here: https://www.rti.com/products/third-party-integrations/nvidia. For any questions, please email [email protected]
This operator requires an installation of [RTI Connext](https://content.rti.com/l/983311/2025-07-08/q5x1n8) to provide access to the DDS domain, as specified by the [OMG Data-Distribution Service](https://www.omg.org/omg-dds-portal/). To obtain a license/activation key, please [request a license/activation key](https://content.rti.com/l/983311/2025-07-25/q6729c). Additional information on Connext can be found [here](https://www.rti.com/products/third-party-integrations/nvidia). For any questions, please email [[email protected]](mailto:[email protected])
🧰 Tools
🪛 markdownlint-cli2 (0.18.1)

8-8: Bare URL used

(MD034, no-bare-urls)


8-8: Bare URL used

(MD034, no-bare-urls)


8-8: Link text should be descriptive

(MD059, descriptive-link-text)

🤖 Prompt for AI Agents
In operators/dds/dds_shapes_subscriber/README.md around line 8, replace the bare
URL and non-descriptive link text: wrap
https://www.rti.com/products/third-party-integrations/nvidia in a markdown link
with descriptive text (e.g., "RTI Connext NVIDIA integrations"), change the
"[click here]" text to something descriptive like "obtain a license/activation
key" and link it to the provided activation URL, and format [email protected]
as a mailto link (mailto:[email protected]) for consistency.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Nov 24, 2025

Greptile Overview

Greptile Summary

Updated DDS Shapes Subscriber documentation to reference "RTI Connext" instead of "RTI Connext Express", added NVIDIA integration resource link, and included support contact email.

  • Changed product name from "RTI Connext Express" to "RTI Connext"
  • Added link to NVIDIA third-party integrations page
  • Added support email contact ([email protected])
  • Email address should be wrapped in backticks per markdown formatting conventions

Confidence Score: 4/5

  • This PR is safe to merge with minimal risk - it only updates documentation text
  • Documentation-only change with straightforward updates to product naming and support information; minor formatting suggestion for email address
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
operators/dds/dds_shapes_subscriber/README.md 4/5 Updated product name from "RTI Connext Express" to "RTI Connext", added NVIDIA integration link and support email; email address should be wrapped in backticks

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant Doc as README.md
    participant User as End User
    
    Dev->>Doc: Update product name
    Note over Doc: Change to "RTI Connext"
    Dev->>Doc: Add NVIDIA integration link
    Dev->>Doc: Add support contact email
    User->>Doc: Read documentation
    Doc-->>User: Updated product info
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

to downstream operators.

This operator requires an installation of [RTI Connext Express](https://content.rti.com/l/983311/2025-07-08/q5x1n8) to provide access to the DDS domain, as specified by the [OMG Data-Distribution Service](https://www.omg.org/omg-dds-portal/). To obtain a license/activation key, please [click here](https://content.rti.com/l/983311/2025-07-25/q6729c). Please see the [usage rules](https://www.rti.com/products/connext-express) for Connext Express.
This operator requires an installation of [RTI Connext](https://content.rti.com/l/983311/2025-07-08/q5x1n8) to provide access to the DDS domain, as specified by the [OMG Data-Distribution Service](https://www.omg.org/omg-dds-portal/). To obtain a license/activation key, please [click here](https://content.rti.com/l/983311/2025-07-25/q6729c). Additional information on Connext can be found here: https://www.rti.com/products/third-party-integrations/nvidia. For any questions, please email [email protected]
Copy link
Contributor

Choose a reason for hiding this comment

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

style: Wrap [email protected] in backticks to prevent platform from interpreting as user mention

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant