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

tools: update user guide (#2098) #2147

Merged
merged 3 commits into from
Apr 2, 2020

Conversation

sre-bot
Copy link
Contributor

@sre-bot sre-bot commented Apr 1, 2020

cherry-pick #2098 to release-3.1


What is changed, added or deleted? (Required)

Update tools user guide according to the latest changes made by Shenli

Note: The English version of Tools User Guide doc is inconsistent from the Chinese version because of this PR. This is known by R&Ds so the subsequent changes to the two docs should be on their own.

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

  • master (the latest development version, including v4.0 changes for now)
  • v3.1 (TiDB 3.1 versions)
  • v3.0 (TiDB 3.0 versions)
  • v2.1 (TiDB 2.1 versions)

**NOTE: I will delete inapplicable content in the 3.1 and 3.0 versions after the cherry-picking. **

If you select two or more versions from above, to trigger the bot to cherry-pick this PR to your desired release version branch(es), you must add corresponding labels such as needs-cherry-pick-3.1, needs-cherry-pick-3.0, and needs-cherry-pick-2.1.

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

@sre-bot sre-bot mentioned this pull request Apr 1, 2020
4 tasks
@sre-bot sre-bot added the type/cherry-pick-for-release-3.1 This PR is cherry-picked to release-3.1 from a source PR. label Apr 1, 2020
@yikeke
Copy link
Contributor

yikeke commented Apr 1, 2020

I deleted content that does not apply to TiDB v3.1 documentation, see this commit. FYI. @lilin90 @WangXiangUSTC

@WangXiangUSTC
Copy link
Contributor

LGTM

@WangXiangUSTC WangXiangUSTC added the status/LGT1 Indicates that a PR has LGTM 1. label Apr 2, 2020
Copy link
Member

@lilin90 lilin90 left a comment

Choose a reason for hiding this comment

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

LGTM

@lilin90 lilin90 merged commit 93d7a01 into pingcap:release-3.1 Apr 2, 2020
@lilin90 lilin90 added the status/LGT2 Indicates that a PR has LGTM 2. label Apr 2, 2020
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/LGT2 Indicates that a PR has LGTM 2. type/cherry-pick-for-release-3.1 This PR is cherry-picked to release-3.1 from a source PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants