Merged
Conversation
…ameter support Update the ToolSet class to provide more flexible parameter handling for list and listAll methods, allowing both legacy and new parameter formats. Also improve API logging with correct request ID header and add example usage in the example file. The changes include: - Refactoring static list and listAll methods to accept parameters in multiple formats - Using listAllResourcesFunction for pagination logic - Updating API logging to use correct x-acs-request-id header - Adding example usage in toolset.ts example file 重构(toolset): 使用灵活的参数支持增强 list 和 listAll 方法 更新 ToolSet 类以提供更灵活的参数处理，支持 list 和 listAll 方法的多种参数格式。同时改进 API 日志记录以使用正确的请求 ID 头部，并在示例文件中添加示例用法。 变更包括： - 重构静态 list 和 listAll 方法以接受多种格式的参数 - 使用 listAllResourcesFunction 进行分页逻辑 - 更新 API 日志以使用正确的 x-acs-request-id 头部 - 在 toolset.ts 示例文件中添加示例用法 Change-Id: I5eb3398fd0120c3674a8c23d81551fdd7c326924 Signed-off-by: OhYee <oyohyee@oyohyee.com>
OhYee
force-pushed
the
fix-list-all-toolsets
branch
from
ec3911d to
86685d4
Compare
Updated test assertions to properly instantiate ToolSet objects instead of using plain objects, ensuring consistent testing of the actual class behavior. 将测试断言更新为正确实例化 ToolSet 对象，而不是使用纯对象， 确保对实际类行为的一致测试。 Change-Id: Ib0761249e6c8255292a4f426ba5ccf2dd428c870 Signed-off-by: OhYee <oyohyee@oyohyee.com>
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
1 participant
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.
Fix bugs
Bug detail
Pull request tasks
Update docs
Reason for update
Pull request tasks
Add contributor
Contributed content
Content detail
Others
Reason for update