Skip to content
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

Create choose-index.md #3079

Merged
merged 25 commits into from
Jul 16, 2020
Merged

Create choose-index.md #3079

merged 25 commits into from
Jul 16, 2020

Conversation

miaoqingli
Copy link
Contributor

What is changed, added or deleted? (Required)

Add choose-index.md

Which TiDB version(s) do your changes apply to? (Required)

  • master (the latest development version)
  • v4.0 (TiDB 4.0 versions)
  • v3.1 (TiDB 3.1 versions)
  • v3.0 (TiDB 3.0 versions)
  • v2.1 (TiDB 2.1 versions)

What is the related PR or file link(s)?

@CLAassistant
Copy link

CLAassistant commented Jun 30, 2020

CLA assistant check
All committers have signed the CLA.

@yikeke yikeke requested review from yikeke and lzmhhh123 June 30, 2020 09:36
@yikeke yikeke added needs-cherry-pick-4.0 status/PTAL This PR is ready for reviewing. translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn. labels Jun 30, 2020
@ti-srebot
Copy link
Contributor

@lzmhhh123, @yikeke, PTAL.

2 similar comments
@ti-srebot
Copy link
Contributor

@lzmhhh123, @yikeke, PTAL.

@ti-srebot
Copy link
Contributor

@lzmhhh123, @yikeke, PTAL.

choose-index.md Outdated Show resolved Hide resolved
choose-index.md Outdated Show resolved Hide resolved
choose-index.md Outdated Show resolved Hide resolved
choose-index.md Outdated Show resolved Hide resolved
choose-index.md Outdated Show resolved Hide resolved
choose-index.md Outdated Show resolved Hide resolved
Copy link
Contributor

@yikeke yikeke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some other minor suggestions. My review is done. You can address the comments now. @miaoqingli

BTW, it's great translations and I'm impressed! Great job!

choose-index.md Outdated Show resolved Hide resolved
choose-index.md Outdated Show resolved Hide resolved
choose-index.md Outdated Show resolved Hide resolved
choose-index.md Outdated Show resolved Hide resolved
choose-index.md Outdated Show resolved Hide resolved
choose-index.md Outdated Show resolved Hide resolved
choose-index.md Outdated Show resolved Hide resolved
choose-index.md Outdated Show resolved Hide resolved
choose-index.md Outdated Show resolved Hide resolved
choose-index.md Outdated Show resolved Hide resolved
@yikeke yikeke added status/require-change Needs the author to address comments. and removed status/PTAL This PR is ready for reviewing. labels Jul 15, 2020
miaoqingli and others added 9 commits July 15, 2020 17:56
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
miaoqingli and others added 7 commits July 15, 2020 18:06
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Copy link
Contributor

@yikeke yikeke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Jul 16, 2020
@yikeke yikeke merged commit 8f1cc59 into pingcap:master Jul 16, 2020
ti-srebot pushed a commit to ti-srebot/docs that referenced this pull request Jul 16, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot ti-srebot mentioned this pull request Jul 16, 2020
5 tasks
@ti-srebot
Copy link
Contributor

cherry pick to release-4.0 in PR #3293

ti-srebot added a commit that referenced this pull request Jul 16, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/LGT1 Indicates that a PR has LGTM 1. status/require-change Needs the author to address comments. translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants