Skip to content

fix(kubevirt): avoid article agreement issue in product name templates#1287

Merged
Cali0707 merged 1 commit into
containers:mainfrom
ksimon1:fix-kubevirt-product-name-article
Jul 13, 2026
Merged

fix(kubevirt): avoid article agreement issue in product name templates#1287
Cali0707 merged 1 commit into
containers:mainfrom
ksimon1:fix-kubevirt-product-name-article

Conversation

@ksimon1

@ksimon1 ksimon1 commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

The vm_clone and vm_create tool descriptions used "a %s VirtualMachine",
which reads fine for "a KubeVirt" but breaks for downstream product name
overrides starting with a vowel sound (e.g. "an OpenShift Virtualization
VirtualMachine"). Rephrase to avoid the leading article before the
templated product name.

/cc @lyarwood

Co-Authored-By: Claude Sonnet 5 noreply@anthropic.com
Signed-off-by: Karel Simon ksimon@redhat.com

The vm_clone and vm_create tool descriptions used "a %s VirtualMachine",
which reads fine for "a KubeVirt" but breaks for downstream product name
overrides starting with a vowel sound (e.g. "an OpenShift Virtualization
VirtualMachine"). Rephrase to avoid the leading article before the
templated product name.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Signed-off-by: Karel Simon <ksimon@redhat.com>
@github-actions

Copy link
Copy Markdown
Contributor

👋 Heads up — this pull request changes files owned by @lyarwood.

You are listed as an owner of one or more of the changed areas in .github/CODEOWNERS. GitHub cannot auto-request review from owners without write access, so this comment is the notification instead. A review when you have a moment would be appreciated 🙏

@Cali0707 Cali0707 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@Cali0707
Cali0707 merged commit b731d56 into containers:main Jul 13, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants