-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
error_code: re-format metafile #8870
Conversation
Signed-off-by: 5kbpers <tangminghua@pingcap.com>
81033aa
to
dd59c4e
Compare
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.
LGTM
(The error code should be KV:xxx:yyy
format, instead of KV-xxx-yyy
)
Signed-off-by: 5kbpers <tangminghua@pingcap.com>
4e3cbc6
to
3ecb234
Compare
Signed-off-by: 5kbpers <tangminghua@pingcap.com>
["KV:Codec:Io"] | ||
error = ''' | ||
KV:Codec:Io | ||
''' |
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.
It is intended that there is a trailing newline?
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.
It's OK, we can trim space when the tiup errdoc component loads them.
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.
LGTM
/merge |
Your auto merge job has been accepted, waiting for:
|
Wow |
It's code freeze day! |
/run-all-tests |
@5kbpers merge failed. |
Seems like a CI environment issue. |
/merge |
Your auto merge job has been accepted, waiting for:
|
/run-all-tests |
@5kbpers merge failed. |
/merge |
/run-all-tests |
@5kbpers merge failed. |
/run-all-tests |
/merge |
/run-all-tests |
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
cherry pick to release-4.0 in PR #8877 |
Signed-off-by: 5kbpers tangminghua@pingcap.com
What is changed and how it works?
Proposal: xxx
What's Changed:
Re-format metafile of error codes.
Related changes
Check List
Tests
Release note