Skip to content

fix: correct grammar in volcano-vgpu userguide running jobs section#337

Closed
mesutoezdil wants to merge 0 commit into
Project-HAMi:masterfrom
mesutoezdil:fix/volcano-vgpu-userguide-grammar
Closed

fix: correct grammar in volcano-vgpu userguide running jobs section#337
mesutoezdil wants to merge 0 commit into
Project-HAMi:masterfrom
mesutoezdil:fix/volcano-vgpu-userguide-grammar

Conversation

@mesutoezdil
Copy link
Copy Markdown
Contributor

Summary

userguide/volcano-vgpu/nvidia-gpu/how-to-use-volcano-vgpu.md line 98:

  • "by both set" corrected to "by setting"
  • Resource names wrapped in backticks
  • resource.limit corrected to resources.limits

Same grammar error as installation/how-to-use-volcano-vgpu.md (fixed separately).

Test plan

  • Verified surrounding context unchanged

@hami-robot hami-robot Bot requested review from archlitchi and wawa0210 May 8, 2026 13:26
@hami-robot hami-robot Bot added the size/XS label May 8, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented May 8, 2026

Deploy Preview for project-hami ready!

Name Link
🔨 Latest commit 9e732f3
🔍 Latest deploy log https://app.netlify.com/projects/project-hami/deploys/69ff87a5cc735e000840bf06
😎 Deploy Preview https://deploy-preview-337--project-hami.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@windsonsea
Copy link
Copy Markdown
Member

/lgtm
/approve

@archlitchi
Copy link
Copy Markdown
Member

please resolve these conflicts

@mesutoezdil mesutoezdil closed this May 9, 2026
@mesutoezdil mesutoezdil force-pushed the fix/volcano-vgpu-userguide-grammar branch from 00ad061 to 9e732f3 Compare May 9, 2026 19:14
@hami-robot hami-robot Bot removed the lgtm label May 9, 2026
@hami-robot
Copy link
Copy Markdown
Contributor

hami-robot Bot commented May 9, 2026

New changes are detected. LGTM label has been removed.

@hami-robot
Copy link
Copy Markdown
Contributor

hami-robot Bot commented May 9, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: mesutoezdil, windsonsea

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hami-robot hami-robot Bot removed the approved label May 9, 2026
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