Skip to content
This repository has been archived by the owner on Aug 21, 2023. It is now read-only.

use cobra to simplify command flags #28

Merged
merged 1 commit into from
Feb 29, 2020

Conversation

siddontang
Copy link
Member

The previous flags handling is ugly, and we can use cobra to simplify them.

@IANTHEREAL
Copy link
Collaborator

CI was broken

@codecov-io
Copy link

Codecov Report

Merging #28 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #28   +/-   ##
=======================================
  Coverage   58.33%   58.33%           
=======================================
  Files          13       13           
  Lines         732      732           
=======================================
  Hits          427      427           
  Misses        265      265           
  Partials       40       40

Copy link
Collaborator

@IANTHEREAL IANTHEREAL left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@tangenta tangenta left a comment

Choose a reason for hiding this comment

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

LGTM

@tangenta tangenta merged commit 7104b68 into pingcap:master Feb 29, 2020
@siddontang siddontang deleted the siddontang/flags branch February 29, 2020 06:40
tisonkun pushed a commit to tisonkun/dumpling that referenced this pull request Oct 20, 2021
tisonkun pushed a commit to tisonkun/dumpling that referenced this pull request Oct 20, 2021
tisonkun pushed a commit to tisonkun/dumpling that referenced this pull request Oct 20, 2021
tisonkun pushed a commit to tisonkun/dumpling that referenced this pull request Oct 20, 2021
tisonkun pushed a commit to tisonkun/tidb that referenced this pull request Oct 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants