milvus-io / milvus-sdk-cpp

C++ SDK for Milvus 2.0
Apache License 2.0
35 stars 21 forks source link

fix build warnings #263

Open Monster880 opened 3 months ago

Monster880 commented 3 months ago

fix build warnings

sre-ci-robot commented 3 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Monster880 To complete the pull request process, please assign matrixji after the PR has been reviewed. You can assign the PR to them by writing /assign @matrixji in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/milvus-io/milvus-sdk-cpp/blob/master/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
sre-ci-robot commented 3 months ago

Welcome @Monster880! It looks like this is your first PR to milvus-io/milvus-sdk-cpp 🎉

Monster880 commented 3 months ago

/assign @matrixji

matrixji commented 3 months ago

Hi, @Monster880 Thanks for your contributions, the clang-format-check failed, you should follow the code-style-for-milvus-c-sdk guide to do clang-format locally before raising PR.