Skip to content

CLI: fixed dead links to tools/main for cli and completion, fixed code owners#17993

Merged
ngxson merged 1 commit intoggml-org:masterfrom
andrew-aladev:feature/fix-dead-links-to-tools-main-cli
Dec 15, 2025
Merged

CLI: fixed dead links to tools/main for cli and completion, fixed code owners#17993
ngxson merged 1 commit intoggml-org:masterfrom
andrew-aladev:feature/fix-dead-links-to-tools-main-cli

Conversation

@andrew-aladev
Copy link
Contributor

@andrew-aladev andrew-aladev commented Dec 13, 2025

Let's start the separation of cli and completion tools.

  1. Fixed dead links for tools/main.
  2. Mentioned cli and completion together where the user can use both tools.
  3. Fixed code owners for cli and completion tools.
  4. Added README.md to tools/cli with TODO.
  5. Used llama-completion instead of llama-cli in tools/completion README.

PR is related to #17824.

I've received an email notification about @jsjtxietian and @bandoti mentioning dead links to CLI and completion. Please review this PR.

Thank you.

/tests/test-chat-.* @pwilkin
/tools/batched-bench/ @ggerganov
/tools/main/ @ggerganov
/tools/cli/ @ngxson
Copy link
Contributor

Choose a reason for hiding this comment

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

@ggerganov should you be added as codeowner of CLI, or it's fine as-is here?

Copy link
Contributor

Choose a reason for hiding this comment

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

I think I'll merge this PR as-is and decide this later if needed

@github-actions github-actions bot added documentation Improvements or additions to documentation examples labels Dec 13, 2025
@ngxson ngxson merged commit 4a4f7e6 into ggml-org:master Dec 15, 2025
2 checks passed
Anico2 added a commit to Anico2/llama.cpp that referenced this pull request Jan 15, 2026
…e owners (ggml-org#17993)

Co-authored-by: Andrew Aladjev <andrew.aladjev@gmail.com>
blime4 referenced this pull request in blime4/llama.cpp Feb 5, 2026
…e owners (#17993)

Co-authored-by: Andrew Aladjev <andrew.aladjev@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation examples

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants