Skip to content

refactor: Remove the requirement of passing content_length to writer#3044

Merged
Xuanwo merged 9 commits intomainfrom
write_can_multig
Sep 13, 2023
Merged

refactor: Remove the requirement of passing content_length to writer#3044
Xuanwo merged 9 commits intomainfrom
write_can_multig

Conversation

@Xuanwo
Copy link
Copy Markdown
Member

@Xuanwo Xuanwo commented Sep 13, 2023

This PR will remove content_length from OpWrite so that users don't need to specify them by hand.

Close #3033

Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
@github-actions github-actions Bot added the releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" label Sep 13, 2023
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo Xuanwo merged commit fda9670 into main Sep 13, 2023
@Xuanwo Xuanwo deleted the write_can_multig branch September 13, 2023 02:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Lack of Test Coverage - API writer for Specific Scenarios

2 participants