Conversation
Contributor
文件级别更改
提示和命令与 Sourcery 互动
自定义您的体验访问您的 dashboard 以:
获取帮助Original review guide in EnglishReviewer's GuideThis PR updates the plugin version, streamlines error logging, extends the command set with /cbl whitelist and /cbl group (including help text and config handling), and aligns the README accordingly. Class Diagram for Plugin and Configuration ChangesclassDiagram
class Plugin {
+VersionNum: Version
-GeneralHooksOnReloadEvent(ReloadEventArgs e) void
#ShowHelpText() void
+HandleCblCommand(CommandArgs args) void
}
class ConfigSettings {
+WhiteList: bool
+GroupNumber: long
+Write() void
}
Plugin --> ConfigSettings : uses
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Contributor
There was a problem hiding this comment.
嘿 @ACaiCat - 我已经审查了你的更改,发现了一些需要解决的问题。
阻塞问题:
- 不能将属性用作 out 参数。(链接)
以下是我在审查期间查看的内容
- 🔴 一般问题: 1 个阻塞问题
- 🟢 安全性: 一切看起来都很好
- 🟢 测试: 一切看起来都很好
- 🟢 文档: 一切看起来都很好
帮助我变得更有用!请点击每个评论上的 👍 或 👎,我将使用反馈来改进你的评论。
Original comment in English
Hey @ACaiCat - I've reviewed your changes and found some issues that need to be addressed.
Blocking issues:
- Cannot use a property as an out parameter. (link)
Here's what I looked at during the review
- 🔴 General issues: 1 blocking issue
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
TUnreal
approved these changes
Jun 5, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
更新插件/修复BUG
其他
好的,这是翻译成中文的 pull request 总结:
Sourcery 总结
添加命令以配置白名单和通知群组,提升版本号,强制完整错误日志记录,并更新文档
新功能:
增强功能:
文档:
Original summary in English
Summary by Sourcery
Add commands to configure whitelist and notification group, bump version, enforce full error logging, and update documentation
New Features:
Enhancements:
Documentation: