-
Notifications
You must be signed in to change notification settings - Fork 24
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
[close #328] cdc: simplify readme #327
Conversation
Signed-off-by: Ping Yu <yuping@pingcap.com>
Signed-off-by: Ping Yu <yuping@pingcap.com>
@zeminzhou @haojinming PTAL, thanks~ |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #327 +/- ##
================================================
- Coverage 61.0409% 60.8734% -0.1676%
================================================
Files 240 240
Lines 20270 20334 +64
================================================
+ Hits 12373 12378 +5
- Misses 6777 6832 +55
- Partials 1120 1124 +4
*This pull request uses carry forward flags. Click here to find out more.
|
/run-integration-test |
3 similar comments
/run-integration-test |
/run-integration-test |
/run-integration-test |
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
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~
Signed-off-by: Ping Yu yuping@pingcap.com
What problem does this PR solve?
Issue Number: close #328
Problem Description: Simplify README
What is changed and how does it work?
Rendered here.
Code changes
Check List for Tests
This PR has been tested by at least one of the following methods:
Side effects
Related changes
To reviewers
Please follow these principles to check this pull requests:
(The above text mainly refers to TiDB Development Guide. It's also highly recommended to read about Writing code review comments)