-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: read configuration #133
Conversation
…ys, and supports memory configuration with units
WalkthroughThe pull request introduces configuration management enhancements in the Kiwi project, focusing on modifying configuration handling in Changes
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
支持读取配置时区分简单字符串和字符串数组, 现在读取配置中带 ' ' 的配置会自动分割,导致一些配置无法正确读取,为后面支持ACL做好准备
支持读取带单位内存配置, 比如 10M 会自动转成 10,485,760 byte
Summary by CodeRabbit
New Features
StringValueArray
class for managing arrays of strings.Bug Fixes
Documentation