Skip to content

Conversation

@czs007
Copy link
Contributor

@czs007 czs007 commented Nov 6, 2024

This PR updates several error messages for grammar and clarity. The changes include:

Corrected "data should be a list of list" to "The data should be a list of lists" for better readability.
Fixed grammar in "The data don't match with schema fields" to "The data doesn't match with schema fields."
Corrected field name mismatch message from "The name of field don't match" to "The name of the field doesn't match."
These improvements enhance the clarity and professionalism of error messages in the codebase.

This PR updates several error messages for grammar and clarity. The
changes include:

Corrected "data should be a list of list" to "The data should be a list
of lists" for better readability.
Fixed grammar in "The data don't match with schema fields" to "The data
doesn't match with schema fields."
Corrected field name mismatch message from "The name of field don't
match" to "The name of the field doesn't match."
These improvements enhance the clarity and professionalism of error
messages in the codebase.

Signed-off-by: zhenshan.cao <[email protected]>
Co-authored-by: Ahmet Yasin Aytar <[email protected]>
@XuanYang-cn
Copy link
Contributor

/lgtm
/approve

@sre-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: czs007, XuanYang-cn

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

The pull request process is described here

Needs approval from an approver in each of these files:

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 sre-ci-robot merged commit c87b21c into milvus-io:2.4 Nov 7, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants