Merged
Conversation
- 修复全局 flag `--url` 与 API 参数 `url` 命名冲突，导致服务器地址被错误作为查询参数发送 - 修复分页参数名错误：API 期望 `page_size`，代码发送了 `size` - 移除 `--config` 全局 flag，配置文件直接使用默认路径加载
- 移除 PersistentPreRun 中 loadConfigFromFile 调用，配置由 main.go ApplyRuntimeConfig 统一传入 - ApplyRuntimeConfig 增加 dryRun 参数，从主命令全局 flag 获取 - 删除已无调用方的 loadConfigFromFile 函数
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
2 participants
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.
--url与 API 参数url命名冲突，导致服务器地址被错误作为查询参数发送page_size，代码发送了size--config全局 flag，配置文件直接使用默认路径加载