-
Notifications
You must be signed in to change notification settings - Fork 663
Open
Labels
corereleases-note/refactorThe PR does a refactor on code or has a title that begins with "refactor"The PR does a refactor on code or has a title that begins with "refactor"
Description
As discussed in #6752
there are still inconsistencies in implementation patterns across various services. This discussion aims to discuss and unify these coding details to improve code consistency and maintainability.
Tasks
- Implement
DebugTrait for all service builders- Most service builders include corresponding service config fields. Special attention should be paid to whether sensitive information might be printed in the debug implementation.
- Module and Structure Renaming
ServiceAccessor→ServiceBackend(to align the naming conventions ofbackendmodule)deletemodule →deletermodule (to align with naming conventions likewriter,reader,lister)
- Remove Empty
listOperation Implementations- Only keep list implementations in services that actually provide this functionality
- Remove useless
listimplementations from services that don't support this operation or have not implemented it yet.
- Unify
SchemeUsage- Prefer direct string-based scheme specification over enum-based approach
- Remove
Schemeenum usage from bindings - Remove the
Schemeenum
- Create a service implementation guide/template
dosubot
Metadata
Metadata
Assignees
Labels
corereleases-note/refactorThe PR does a refactor on code or has a title that begins with "refactor"The PR does a refactor on code or has a title that begins with "refactor"