Skip to content

refactor: 将 ClearHitCount 的 strict 默认值改为 false#1081

Merged
MistEO merged 1 commit intoMaaEnd:mainfrom
ocsin1:ref/ClearHitCount
Mar 7, 2026
Merged

refactor: 将 ClearHitCount 的 strict 默认值改为 false#1081
MistEO merged 1 commit intoMaaEnd:mainfrom
ocsin1:ref/ClearHitCount

Conversation

@ocsin1
Copy link
Contributor

@ocsin1 ocsin1 commented Mar 7, 2026

Summary by Sourcery

将 ClearHitCount 自定义操作更改为默认使用非严格模式,并更新文档以匹配新行为。

增强内容:

  • 调整 ClearHitCount 操作,使 strict 参数默认值从 true 改为 false,这样在未显式启用严格模式时,部分失败将不会被视为致命错误。

文档:

  • 更新英文和中文的自定义操作文档,以说明 strict 参数的新默认值,并澄清在启用严格模式时的具体行为。
Original summary in English

Summary by Sourcery

Change the ClearHitCount custom action to use non-strict mode by default and update documentation to match the new behavior.

Enhancements:

  • Adjust ClearHitCount action so the strict parameter defaults to false instead of true, making partial failures non-fatal unless strict is explicitly enabled.

Documentation:

  • Update English and Chinese custom action documentation to describe the new default for the strict parameter and clarify behavior when strict mode is enabled.

Copy link
Contributor

@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.

@MistEO MistEO merged commit e636d3c into MaaEnd:main Mar 7, 2026
17 checks passed
@ocsin1 ocsin1 deleted the ref/ClearHitCount branch March 7, 2026 16:27
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