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

support using task-file when need task-name (#257) #258

Merged
merged 7 commits into from
Aug 19, 2020

Conversation

ti-srebot
Copy link
Contributor

cherry-pick #257 to release-1.0


What is changed, added or deleted? (Required)

support using task-file when need task-name

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

  • master (the latest development version, including v2.0 changes for now)
  • v1.0 (TiDB DM 1.0 versions)

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

Do your changes match any of the following descriptions?

  • Delete files
  • Change aliases
  • Have version specific changes
  • Might cause conflicts

Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot ti-srebot added size/small Changes of a small size. type/cherry-pick-for-release-1.0 This PR is cherry-picked to release-1.0 from a source PR. v1.0 This PR/issue applies to TiDB DM v1.0. v2.0 This PR/issue applies to TiDB DM v2.0. requires-version-specific-changes After cherry-picked, the cherry-picked PR requires further changes. labels Aug 19, 2020
@ti-srebot
Copy link
Contributor Author

@Joyinqin please accept the invitation then you can push to the cherry-pick pull requests.
https://github.com/ti-srebot/docs-dm/invitations

@lilin90
Copy link
Member

lilin90 commented Aug 19, 2020

I'm working on this.

@lilin90
Copy link
Member

lilin90 commented Aug 19, 2020

@lance6716 PTAL

Copy link
Contributor

@lance6716 lance6716 left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -14,6 +14,10 @@ The dmctl component supports the interactive mode for manual operations, and als

This section describes the basic use of dmctl commands in the interactive mode.

> **Note:**
>
> The interactive mode does not support Bash features. For example, you need to directly pass string flags instead of passing them in quotation marks.
Copy link
Contributor

Choose a reason for hiding this comment

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

seems coders casually say "quotes" instead of "quotation marks"

Copy link
Member

Choose a reason for hiding this comment

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

@lance6716 Updated. Thanks~

@ti-srebot
Copy link
Contributor Author

@lance6716,Thanks for your review. However, LGTM is restricted to Reviewers or higher roles.See the corresponding SIG page for more information. Related SIGs: docs(slack).

1 similar comment
@ti-srebot
Copy link
Contributor Author

@lance6716,Thanks for your review. However, LGTM is restricted to Reviewers or higher roles.See the corresponding SIG page for more information. Related SIGs: docs(slack).

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

@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Aug 19, 2020
@lilin90
Copy link
Member

lilin90 commented Aug 19, 2020

/merge

@ti-srebot ti-srebot added the status/can-merge Indicates a PR has been approved by a committer. label Aug 19, 2020
@lilin90 lilin90 merged commit d6adc27 into pingcap:release-1.0 Aug 19, 2020
@lance6716 lance6716 mentioned this pull request Aug 19, 2020
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
requires-version-specific-changes After cherry-picked, the cherry-picked PR requires further changes. size/small Changes of a small size. status/can-merge Indicates a PR has been approved by a committer. status/LGT1 Indicates that a PR has LGTM 1. type/cherry-pick-for-release-1.0 This PR is cherry-picked to release-1.0 from a source PR. v1.0 This PR/issue applies to TiDB DM v1.0. v2.0 This PR/issue applies to TiDB DM v2.0.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants