-
Notifications
You must be signed in to change notification settings - Fork 70
use crate TypedBuilder for builders #184
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
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR migrates from hand-rolled builder structs to the TypedBuilder
crate, implementing a standardized builder pattern across the entire S3 client library. This change introduces the .build()
method call requirement for all request builders while maintaining the existing API surface.
- Adopts
TypedBuilder
for consistent, type-safe builder patterns across all S3 operations - Updates all client methods to return
TypedBuilder
-generated builder types - Adds required
.build()
calls to all test files for new builder API
Reviewed Changes
Copilot reviewed 204 out of 204 changed files in this pull request and generated 6 comments.
File | Description |
---|---|
test files | Updated all test method calls to include required .build() step |
client files | Modified all client methods to return TypedBuilder-generated builders |
types.rs | Updated S3Request to use TypedBuilder with proper field annotations |
response files | Updated documentation references from Client to MinioClient |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
303c43e
to
f7396a9
Compare
119def6
to
55acda0
Compare
bugfixes WIP doc updated
c855dfe
to
2c6f943
Compare
This PR migrates from hand-rolled builder structs to the TypedBuilder crate, implementing a standardized builder pattern across the entire S3 client library. This change introduces the .build() method call requirement for all request builders while maintaining the existing API surface.
Adopts TypedBuilder for consistent, type-safe builder patterns across all S3 operations
Updates all client methods to return TypedBuilder-generated builder types
Adds required .build() calls to all test files for new builder API