Skip to content

docs: fix ray initialization comment typo#10628

Merged
hiyouga merged 1 commit into
hiyouga:mainfrom
kmishra1204:codex/fix-ray-comment-typo
Jul 7, 2026
Merged

docs: fix ray initialization comment typo#10628
hiyouga merged 1 commit into
hiyouga:mainfrom
kmishra1204:codex/fix-ray-comment-typo

Conversation

@kmishra1204

Copy link
Copy Markdown
Contributor

What does this PR do?

Fixes a typo in a Ray cleanup comment in src/llamafactory/train/tuner.py.

This is a docs/comment-only cleanup; runtime behavior is unchanged.

Before submitting

  • Did you read the contributor guideline?
  • Did you write any new necessary tests? N/A, comment-only typo fix.

Validation

  • uvx --from codespell codespell src\\llamafactory\\train\\tuner.py
  • uvx ruff check src\\llamafactory\\train\\tuner.py
  • git diff --check

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request fixes a typo in a comment within src/llamafactory/train/tuner.py, correcting 'intialized' to 'initialized'. There are no review comments, and I have no feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@kmishra1204

Copy link
Copy Markdown
Contributor Author

Validated after a fresh fetch from upstream main.

  • Branch: codex/fix-ray-comment-typo
  • Status: 1 commit ahead, 0 behind upstream main
  • Scope: src/llamafactory/train/tuner.py
  • Checks run locally:
    • git diff --check origin/main...fork/codex/fix-ray-comment-typo
    • uv run --no-project --with ruff -- ruff check src/llamafactory/train/tuner.py

Both checks passed. No project dependency install or Docker run was used.

@hiyouga hiyouga added the solved This problem has been already solved label Jul 7, 2026
@hiyouga hiyouga merged commit 76a0391 into hiyouga:main Jul 7, 2026
14 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

solved This problem has been already solved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants