Skip to content

feat(bindings/C): Add opendal_operator_rename and opendal_operator_copy#3517

Merged
xyjixyjixyji merged 4 commits intoapache:mainfrom
jiaoew1991:add-more-apis-c
Nov 8, 2023
Merged

feat(bindings/C): Add opendal_operator_rename and opendal_operator_copy#3517
xyjixyjixyji merged 4 commits intoapache:mainfrom
jiaoew1991:add-more-apis-c

Conversation

@jiaoew1991
Copy link
Copy Markdown
Contributor

No description provided.

@suyanhanx suyanhanx changed the title Add opendal_operator_rename and opendal_operator_copy for C api feat(bindings/c): Add opendal_operator_rename and opendal_operator_copy Nov 8, 2023
@github-actions github-actions Bot added the releases-note/feat The PR implements a new feature or has a title that begins with "feat" label Nov 8, 2023
Copy link
Copy Markdown
Contributor

@xyjixyjixyji xyjixyjixyji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! The code and test LGTM. But I think documentations need to be added as well.

@jiaoew1991
Copy link
Copy Markdown
Contributor Author

Thanks! The code and test LGTM. But I think documentations need to be added as well.

fixed it 😊

Copy link
Copy Markdown
Contributor

@xyjixyjixyji xyjixyjixyji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@xyjixyjixyji xyjixyjixyji changed the title feat(bindings/c): Add opendal_operator_rename and opendal_operator_copy feat(bindings/C): Add opendal_operator_rename and opendal_operator_copy Nov 8, 2023
@xyjixyjixyji xyjixyjixyji merged commit 880b648 into apache:main Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/feat The PR implements a new feature or has a title that begins with "feat"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants