-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
CONTRIBUTING: '/' -> '/' #8954
CONTRIBUTING: '/' -> '/' #8954
Conversation
Codecov Report
@@ Coverage Diff @@
## master #8954 +/- ##
==========================================
- Coverage 67.53% 67.52% -0.02%
==========================================
Files 363 363
Lines 75199 75199
==========================================
- Hits 50787 50778 -9
- Misses 19931 19936 +5
- Partials 4481 4485 +4
Continue to review full report at Codecov.
|
@@ -15,7 +15,7 @@ A Contributor refers to the person who contributes to the following projects: | |||
|
|||
## How to become a TiDB Contributor? | |||
|
|||
If a PR (Pull Request) submitted to the TiDB / TiKV / TiSpark / PD / Docs/Docs-cn projects by you is approved and merged, then you become a TiDB Contributor. |
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.
I think "TiDB/TiKV/TiSpark/PD/Docs/Docs-cn" is better.
@queenypingcap PTAL
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.
I agree
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
What problem does this PR solve?
Improve documentation.
What is changed and how it works?
/
->/
inCONTRIBUTING.md
Check List
Tests
Code changes
Side effects
Related changes
This change is