Skip to content

Conversation

@JasonHuang79
Copy link
Contributor

@JasonHuang79 JasonHuang79 commented Jan 6, 2026

Add usage tips for Windows package manager users.

Summary by Sourcery

文档:

  • 在中文使用文档中添加一个提示,说明使用 winget 安装时,在示例中应使用 maa-cli 作为命令名称,而不是 maa
Original summary in English

Summary by Sourcery

Documentation:

  • Add a tip in the Chinese usage documentation explaining that winget installations should use the maa-cli command name instead of maa in examples.
- 新增提示,说明通过 winget 安装的用户在参考文档示例时,应使用命令名 `maa-cli` 而不是 `maa`。
Original summary in English

Summary by Sourcery

文档:

  • 在中文使用文档中添加一个提示,说明使用 winget 安装时,在示例中应使用 maa-cli 作为命令名称,而不是 maa
Original summary in English

Summary by Sourcery

Documentation:

  • Add a tip in the Chinese usage documentation explaining that winget installations should use the maa-cli command name instead of maa in examples.

Add usage tips for Windows package manager users.
Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

你好——我在这里给出了一些整体性的反馈:

  • 中文提示这句话语法上有点别扭;可以考虑改写为 “对于使用 Windows 包管理器 (winget) 安装 maa-cli 的用户,以下命令中的 maa 需要替换为 maa-cli。” 来让读起来更流畅。
供 AI Agent 使用的提示词
Please address the comments from this code review:

## Overall Comments
- The Chinese tip sentence is slightly awkward grammatically; consider rephrasing to something like “对于使用 Windows 包管理器 (winget) 安装 maa-cli 的用户,以下命令中的 `maa` 需要替换为 `maa-cli`。” for smoother readability.

Sourcery 对开源项目免费——如果你觉得我们的代码评审有帮助,欢迎分享 ✨
帮我变得更有用!请在每条评论上点 👍 或 👎,我会根据你的反馈改进后续的代码评审。
Original comment in English

Hey - I've left some high level feedback:

  • The Chinese tip sentence is slightly awkward grammatically; consider rephrasing to something like “对于使用 Windows 包管理器 (winget) 安装 maa-cli 的用户,以下命令中的 maa 需要替换为 maa-cli。” for smoother readability.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- The Chinese tip sentence is slightly awkward grammatically; consider rephrasing to something like “对于使用 Windows 包管理器 (winget) 安装 maa-cli 的用户,以下命令中的 `maa` 需要替换为 `maa-cli`。” for smoother readability.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@JasonHuang79
Copy link
Contributor Author

@wangl-cc 您看一下这里,因为命令有变,所以我加了个提示。
但是我不知道这个容器的格式是不是对的,我直接套的 gui 那边文档的容器格式。

@JasonHuang79
Copy link
Contributor Author

@sourcery-ai title

@sourcery-ai sourcery-ai bot changed the title docs: Add a tip for who use winget install maa-cli Add docs tip for winget users to use maa-cli command name Jan 6, 2026
@JasonHuang79 JasonHuang79 changed the title Add docs tip for winget users to use maa-cli command name docs: Add docs tip for winget users to use maa-cli command name Jan 6, 2026
@JasonHuang79
Copy link
Contributor Author

@sourcery-ai review

Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

嗨——我已经审阅了你的修改,看起来非常棒!


Sourcery 对开源项目是免费的——如果你觉得我们的评审有帮助,欢迎分享 ✨
帮我变得更有用!请在每条评论上点 👍 或 👎,我会根据你的反馈改进以后的评审。
Original comment in English

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

对于使用 Windows 包管理器 (winget) 安装 maa-cli 的用户,以下命令中的 `maa` 需要替换为 `maa-cli`

:::

Copy link
Member

Choose a reason for hiding this comment

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

这一行有一个空格,可能要删掉。


对于使用 Windows 包管理器 (winget) 安装 maa-cli 的用户,以下命令中的 `maa` 需要替换为 `maa-cli`

:::
Copy link
Member

Choose a reason for hiding this comment

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

这个其实是纯 Markdown,所以不支持容器,你可以直接不用容器。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

行,但是主仓库那边是支持容器的,所以我有点好奇这边上游的仓库的文档支不支持容器

@wangl-cc
Copy link
Member

wangl-cc commented Jan 7, 2026

我在考虑要不要单独维护 CLI 这边的文档,因为两边同步起来比较麻烦。我可以先把这个合了。然后你在主仓库也可以提个 PR。后续改文档都直接提到主仓库吧。

@JasonHuang79
Copy link
Contributor Author

我在考虑要不要单独维护 CLI 这边的文档,因为两边同步起来比较麻烦。我可以先把这个合了。然后你在主仓库也可以提个 PR。后续改文档都直接提到主仓库吧。

我记得没错的话,主仓库的文档是同步这个仓库的文档的吧?
那我之后直接提到主仓库试试

@wangl-cc
Copy link
Member

wangl-cc commented Jan 7, 2026

之前是这样的,但是主仓库现在做了很多修改,可能要经常反向同步,所以不如只维护那边了。

@wangl-cc wangl-cc merged commit 8fbcfc2 into MaaAssistantArknights:main Jan 7, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants