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

.github: update the label for sigs #19125

Merged
merged 2 commits into from
Aug 11, 2020
Merged

Conversation

AndreMouche
Copy link
Contributor

Signed-off-by: shirly AndreMouche@126.com

What problem does this PR solve?

This PR update the labels for sigs

What is changed and how it works?

auto label sig's label when PR created

Related changes

no

Check List

no

Release note

No release note

Signed-off-by: shirly <AndreMouche@126.com>
@AndreMouche AndreMouche requested review from zz-jason, SunRunAway and zimulala and removed request for SunRunAway August 11, 2020 06:26
Copy link
Member

@zz-jason zz-jason 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 11, 2020
Signed-off-by: shirly <AndreMouche@126.com>
Copy link
Contributor

@qw4990 qw4990 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 status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Aug 11, 2020
@AndreMouche
Copy link
Contributor Author

/merge

@ti-srebot ti-srebot added the status/can-merge Indicates a PR has been approved by a committer. label Aug 11, 2020
@ti-srebot
Copy link
Contributor

/run-all-tests

@codecov
Copy link

codecov bot commented Aug 11, 2020

Codecov Report

Merging #19125 into master will decrease coverage by 0.0539%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             master     #19125        +/-   ##
================================================
- Coverage   79.0943%   79.0404%   -0.0540%     
================================================
  Files           547        547                
  Lines        148606     148548        -58     
================================================
- Hits         117539     117413       -126     
- Misses        21603      21652        +49     
- Partials       9464       9483        +19     

Copy link
Contributor

@zimulala zimulala 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 status/LGT3 The PR has already had 3 LGTM. and removed status/LGT2 Indicates that a PR has LGTM 2. labels Aug 11, 2020
@ti-srebot ti-srebot merged commit 2de3aed into pingcap:master Aug 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/can-merge Indicates a PR has been approved by a committer. status/LGT3 The PR has already had 3 LGTM.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants