Rename "Insights Advisor" to "Red Hat Lightspeed Advisor" in API docs#2911
Rename "Insights Advisor" to "Red Hat Lightspeed Advisor" in API docs#2911Jakub007d wants to merge 1 commit into
Conversation
|
Pipeline controller notification For optional jobs, comment This repository is configured in: LGTM mode |
|
Hello @Jakub007d! Some important instructions when contributing to openshift/api: |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Hi @Jakub007d. Thanks for your PR. I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with Regular contributors should join the org to skip this step. Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Enterprise Run ID: ⛔ Files ignored due to path filters (13)
📒 Files selected for processing (5)
📝 WalkthroughWalkthroughThis pull request updates documentation comments across multiple Go type definition files (insights v1, v1alpha1, v1alpha2, and operator v1) and a corresponding CRD manifest file. The change replaces references to "Insights Advisor" with "Red Hat Lightspeed Advisor" in field descriptions for the Changes
Sequence Diagram(s)Not applicable; this change is limited to documentation/comment text updates with no behavioral or interaction changes. Estimated code review effort: 1 (Low) — purely textual documentation changes with no logic, type, or validation modifications. Related issues: None specified. Related PRs: None specified. Suggested labels: documentation, low-risk Suggested reviewers: None specified. 🐰 A rabbit hopped through comments neat, 🚥 Pre-merge checks | ✅ 15✅ Passed checks (15 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 golangci-lint (2.12.2)Error: build linters: unable to load custom analyzer "kubeapilinter": tools/_output/bin/kube-api-linter.so, plugin: not implemented Comment |
Rename all "Insights Advisor" references in API field descriptions to "Red Hat Lightspeed Advisor" across operator/v1, insights/v1, insights/v1alpha1, and insights/v1alpha2 types.
The generated swagger docs, CRD manifests, and OpenAPI specs were updated with
make update.Ref: https://redhat.atlassian.net/browse/HCCDOC-4522