Skip to content

Conversation

@yuva29
Copy link
Collaborator

@yuva29 yuva29 commented Jan 23, 2026

No description provided.

@yuva29 yuva29 requested a review from Copilot January 23, 2026 22:46
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the wording in the Kubernetes Helm installation documentation to improve clarity when directing users to information about installing multiple operators together.

Changes:

  • Updated phrasing in the warning message from "jump to this section" to "refer" for better clarity

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


> **Warning**
> If you are trying to install AMD GPU Operator and AMD Network Operator together within the same Kubernetes cluster, please jump to this section [Installation of GPU Operator and Network Operator together](./kubernetes-helm-operators.md)
> If you are trying to install AMD GPU Operator and AMD Network Operator together within the same Kubernetes cluster, please refer [Installation of GPU Operator and Network Operator together](./kubernetes-helm-operators.md)
Copy link

Copilot AI Jan 23, 2026

Choose a reason for hiding this comment

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

The phrase 'please refer' is incomplete. It should be 'please refer to' to be grammatically correct.

Suggested change
> If you are trying to install AMD GPU Operator and AMD Network Operator together within the same Kubernetes cluster, please refer [Installation of GPU Operator and Network Operator together](./kubernetes-helm-operators.md)
> If you are trying to install AMD GPU Operator and AMD Network Operator together within the same Kubernetes cluster, please refer to [Installation of GPU Operator and Network Operator together](./kubernetes-helm-operators.md)

Copilot uses AI. Check for mistakes.
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sajmera-pensando sajmera-pensando merged commit 6bd2536 into ROCm:main Jan 23, 2026
2 checks passed
@yuva29 yuva29 deleted the docs branch January 23, 2026 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants