Skip to content
This repository was archived by the owner on Jan 15, 2026. It is now read-only.

Relax LGTMs#197

Merged
coolljt0725 merged 1 commit intomasterfrom
relax-pullapprove
Jan 24, 2018
Merged

Relax LGTMs#197
coolljt0725 merged 1 commit intomasterfrom
relax-pullapprove

Conversation

@caniszczyk
Copy link
Copy Markdown
Contributor

There has been a request to relax the number of LGTMs in the
tools projects to help improve merge rate.

Signed-off-by: Chris Aniszczyk caniszczyk@gmail.com

There has been a request to relax the number of LGTMs in the
tools projects to help improve merge rate.

Signed-off-by: Chris Aniszczyk <caniszczyk@gmail.com>
@zhouhao3
Copy link
Copy Markdown

zhouhao3 commented Jan 24, 2018

LGTM

Approved with PullApprove

1 similar comment
@coolljt0725
Copy link
Copy Markdown
Member

coolljt0725 commented Jan 24, 2018

LGTM

Approved with PullApprove

@coolljt0725 coolljt0725 merged commit fa160db into master Jan 24, 2018
@stevvooe
Copy link
Copy Markdown
Contributor

I think it requires more than 2 LGTMs to modify the operational rules of the project. Was there a governance vote on this?

@stevvooe stevvooe deleted the relax-pullapprove branch February 6, 2018 19:06
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